drgreen/android_frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
android_frameworks_base/config/OWNERS

15 lines
635 B
Plaintext
Raw Normal View History

Improve OWNERS coverage across frameworks/base/. 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. Some teams under frameworks/base/ have been using OWNERS files successfully for many years, and we're ready to expand them to cover more areas. Here's the historical coverage statistics for the last two years of changes before these new OWNERS changes land: -- 56% of changes are fully covered by OWNERS -- 17% of changes are partially covered by OWNERS -- 25% of changes have no OWNERS coverage Working closely with team leads, we've now identified clear OWNERS on a per-package basis, and we're using "include" directives whenever possible to to simplify future maintenance. With this extensive effort, we've now improved our coverage as follows: -- 98% of changes are fully covered by OWNERS -- 1% of changes are partially covered by OWNERS -- 1% of changes have no OWNERS coverage This specific change is automatically generated by a script that identifies relevant "include" directives. Bug: 174932174 Test: manual Exempt-From-Owner-Approval: refactoring with team leads buy-in Merged-In: I3480ddf2fe7ba3dfb922b459d4da01fa17a2c813 Change-Id: I3480ddf2fe7ba3dfb922b459d4da01fa17a2c813
2020-12-08 08:36:17 -07:00
include /ZYGOTE_OWNERS
Add OWNERS for frameworks/base/config/hiddenapi*. Test: N/A Change-Id: Iaf914899686ea6e8be7462f0c1df0646c4fa69b0 Merged-In: Iaf914899686ea6e8be7462f0c1df0646c4fa69b0
2019-11-19 17:41:03 +00:00
# compat-team@ for changes to hiddenapi files
Improve OWNERS coverage across frameworks/base/. 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. Some teams under frameworks/base/ have been using OWNERS files successfully for many years, and we're ready to expand them to cover more areas. Here's the historical coverage statistics for the last two years of changes before these new OWNERS changes land: -- 56% of changes are fully covered by OWNERS -- 17% of changes are partially covered by OWNERS -- 25% of changes have no OWNERS coverage Working closely with team leads, we've now identified clear OWNERS on a per-package basis, and we're using "include" directives whenever possible to to simplify future maintenance. With this extensive effort, we've now improved our coverage as follows: -- 98% of changes are fully covered by OWNERS -- 1% of changes are partially covered by OWNERS -- 1% of changes have no OWNERS coverage This specific change is automatically generated by a script that identifies relevant "include" directives. Bug: 174932174 Test: manual Exempt-From-Owner-Approval: refactoring with team leads buy-in Merged-In: I3480ddf2fe7ba3dfb922b459d4da01fa17a2c813 Change-Id: I3480ddf2fe7ba3dfb922b459d4da01fa17a2c813
2020-12-08 08:36:17 -07:00
Remove atrost (and dbrazdil) from OWNERS files. Test: none Change-Id: If82422229d1b4cbed5e72d1c9042bd6a38be54e5
2020-05-12 12:36:02 +01:00
per-file hiddenapi-* = andreionea@google.com, mathewi@google.com, satayev@google.com
Add OWNERS for frameworks/base/config/hiddenapi*. Test: N/A Change-Id: Iaf914899686ea6e8be7462f0c1df0646c4fa69b0 Merged-In: Iaf914899686ea6e8be7462f0c1df0646c4fa69b0
2019-11-19 17:41:03 +00:00
Update boot image profiles OWNERS Test: m Change-Id: Ic4190cba73d40621a82ea818051d40e6b578357b
2021-01-22 17:53:18 -08:00
# art-team@ manages the boot image profiles
per-file boot-* = calin@google.com, mathieuc@google.com, ngeoffray@google.com
per-file dirty-image-objects = calin@google.com, mathieuc@google.com, ngeoffray@google.com
per-file generate-preloaded-classes.sh = calin@google.com, mathieuc@google.com, ngeoffray@google.com
per-file preloaded-classes* = calin@google.com, mathieuc@google.com, ngeoffray@google.com
Add OWNERS for frameworks/base/config/hiddenapi*. Test: N/A Change-Id: Iaf914899686ea6e8be7462f0c1df0646c4fa69b0 Merged-In: Iaf914899686ea6e8be7462f0c1df0646c4fa69b0
2019-11-19 17:41:03 +00:00
# Escalations:
Remove atrost (and dbrazdil) from OWNERS files. Test: none Change-Id: If82422229d1b4cbed5e72d1c9042bd6a38be54e5
2020-05-12 12:36:02 +01:00
per-file hiddenapi-* = bdc@google.com, narayan@google.com
Reference in New Issue Copy Permalink