diff options
author | Jeff Sharkey <jsharkey@android.com> | 2020-12-08 15:57:15 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2020-12-08 15:57:15 +0000 |
commit | b84eaab33192da65c95a954f9859159186789709 (patch) | |
tree | 8f7055228bb5234beecf6e45208872607da45057 | |
parent | 5d775cd03482eb4d77f507e8d6ca46043b0905c3 (diff) | |
parent | 061c2ec3b61fb2d62be2c46dee9696e5710dc55f (diff) | |
download | base-b84eaab33192da65c95a954f9859159186789709.tar.gz |
Improve OWNERS coverage across frameworks/base/. am: 061c2ec3b6
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1519382
MUST ONLY BE SUBMITTED BY AUTOMERGER
Change-Id: I1218107820d89a570b55768ed6f9b223cd0db5fc
-rw-r--r-- | OWNERS | 19 | ||||
-rw-r--r-- | OWNERS.md | 34 | ||||
-rw-r--r-- | ZYGOTE_OWNERS | 5 | ||||
-rw-r--r-- | core/api/OWNERS | 2 | ||||
-rw-r--r-- | core/res/OWNERS | 17 | ||||
-rw-r--r-- | data/etc/OWNERS | 14 | ||||
-rw-r--r-- | services/api/OWNERS | 2 |
7 files changed, 92 insertions, 1 deletions
diff --git a/OWNERS b/OWNERS new file mode 100644 index 000000000000..9b0a8bdb3709 --- /dev/null +++ b/OWNERS @@ -0,0 +1,19 @@ +# This top-level list should remain narrowly defined as team leads; individual +# teams are strongly encouraged to define narrower OWNERS files at deeper +# levels within the source tree; see OWNERS.md for more details +dsandler@android.com +dsandler@google.com +hackbod@android.com +hackbod@google.com +jsharkey@android.com +jsharkey@google.com +michaelwr@google.com +nandana@google.com +narayan@google.com +ogunwale@google.com +svetoslavganov@android.com +svetoslavganov@google.com +yamasani@google.com + +# Support bulk translation updates +per-file */res*/values*/*.xml = byi@google.com diff --git a/OWNERS.md b/OWNERS.md new file mode 100644 index 000000000000..6428c59fd793 --- /dev/null +++ b/OWNERS.md @@ -0,0 +1,34 @@ +As general background, `OWNERS` files expedite code reviews by helping code +authors quickly find relevant reviewers, and they also ensure that stakeholders +are involved in code changes in their areas. + +The structure of `frameworks/base/` is unique among Android repositories, and +it's evolved into a complex interleaved structure over the years. Because of +this structure, the best place to authoritatively define `OWNERS` can vary +wildly, but here are some common patterns: + +* `core/java/` contains source that is included in the base classpath, and as +such it's where most APIs are defined: + * `core/java/android/app/` + * `core/java/android/content/` +* `services/core/` contains most system services, and these directories +typically have more granularity than `core/java/`, since they can be refactored +without API changes: + * `services/core/java/com/android/server/net/` + * `services/core/java/com/android/server/wm/` +* `services/` contains several system services that have been isolated from the +main `services/core/` project: + * `services/appwidget/` + * `services/midi/` +* `apex/` contains Mainline modules: + * `apex/jobscheduler/` + * `apex/permission/` +* Finally, some teams may have dedicated top-level directories: + * `media/` + * `wifi/` + +Area maintainers are strongly encouraged to list people in a single +authoritative `OWNERS` file in **exactly one** location. Then, other paths +should reference that single authoritative `OWNERS` file using an include +directive. This approach ensures that updates are applied consistently across +the tree, reducing maintenance burden. diff --git a/ZYGOTE_OWNERS b/ZYGOTE_OWNERS new file mode 100644 index 000000000000..90a185b79b52 --- /dev/null +++ b/ZYGOTE_OWNERS @@ -0,0 +1,5 @@ +calin@google.com +chriswailes@google.com +maco@google.com +narayan@google.com +ngeoffray@google.com diff --git a/core/api/OWNERS b/core/api/OWNERS new file mode 100644 index 000000000000..0b95c5155594 --- /dev/null +++ b/core/api/OWNERS @@ -0,0 +1,2 @@ +# API changes are managed via Prolog rules, not OWNERS +* diff --git a/core/res/OWNERS b/core/res/OWNERS new file mode 100644 index 000000000000..263d638353d6 --- /dev/null +++ b/core/res/OWNERS @@ -0,0 +1,17 @@ +adamp@google.com +alanv@google.com +dsandler@android.com +dsandler@google.com +hackbod@android.com +hackbod@google.com +jsharkey@android.com +jsharkey@google.com +michaelwr@google.com +nandana@google.com +narayan@google.com +ogunwale@google.com +patb@google.com +svetoslavganov@android.com +svetoslavganov@google.com +toddke@google.com +yamasani@google.com diff --git a/data/etc/OWNERS b/data/etc/OWNERS index 70d467829269..5efd0bd06b74 100644 --- a/data/etc/OWNERS +++ b/data/etc/OWNERS @@ -1 +1,13 @@ -per-file privapp-permissions-platform.xml = hackbod@android.com, jsharkey@android.com, svetoslavganov@google.com, toddke@google.com, yamasani@google.com, cbrubaker@google.com, jeffv@google.com, moltmann@google.com, lorenzo@google.com +cbrubaker@google.com +hackbod@android.com +hackbod@google.com +jeffv@google.com +jsharkey@android.com +jsharkey@google.com +lorenzo@google.com +moltmann@google.com +svetoslavganov@android.com +svetoslavganov@google.com +toddke@android.com +toddke@google.com +yamasani@google.com diff --git a/services/api/OWNERS b/services/api/OWNERS new file mode 100644 index 000000000000..0b95c5155594 --- /dev/null +++ b/services/api/OWNERS @@ -0,0 +1,2 @@ +# API changes are managed via Prolog rules, not OWNERS +* |