Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Remove inactive owners from /chrome/browser/ui/app_list/search/cros_a…
…ction_history Per discussion on the chromium-dev@ thread [1], we remove owners who made 0 commits / reviews to the Chromium repository in the past 12 months (2021/7/26 - 2022/7/26). For each owner removal, we upload a CL to remove the owner. The CL adds the owner being removed and two other owners as reviewers. The CL will be committed once it gets an LGTM from one of the reviewers. Instructions: - The guiding principle is: only owners who are meeting the owner's expectations [2] should be written in OWNERS files. This is important to ensure that developers can find active owners by looking at the OWNERS files. - Please LGTM or not-LGTM the CL based on the removal guideline [3]. i.e., if the owner being removed was inactive only for a certain period of time while they were on a long leave and has been meeting the owner's expectations in other times, please not-LGTM the CL. Otherwise, please LGTM the CL. - If the CL was LGTMed and landed by mistake (e.g., the reviewer was not aware of the owner's leave status), we can just revert the CL. - If you have any questions / comments, please send an explicit email to [email protected]. (haraken@ is not watching all the comments of all the removal CLs :-) If there is any disagreement between owners, it will be escalated to Chrome Eng Review and resolved per [3]. [1] https://groups.google.com/a/chromium.org/g/chromium-dev/c/aGC8BTTFK64 [2] https://chromium.googlesource.com/chromium/src/+/main/docs/code_reviews.md#expectations-of-owners [3] https://chromium.googlesource.com/chromium/src/+/main/docs/code_reviews.md#removal-of-owners This CL was uploaded by git cl split. [email protected], [email protected], [email protected] Bug: 1355150 Change-Id: I97603d01d1e7a882c9f54bca02b7deddb0f51265 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3844665 Auto-Submit: Kentaro Hara <[email protected]> Commit-Queue: Tony Yeoman <[email protected]> Reviewed-by: Tony Yeoman <[email protected]> Cr-Commit-Position: refs/heads/main@{#1038011}
- Loading branch information