-
Notifications
You must be signed in to change notification settings - Fork 499
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move Bazel ownership to end of codeowners file #3120
Conversation
@rt4914 PTAL for codeowner adjustment. @fsharpasharp PTAL for policy change in how we manage Bazel owners. @seanlip PTAL as codeowner owner. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry -- somehow I missed this notification. LGTM but there's a merge conflict that needs to be resolved.
Undo post-merge conflict mistake.
Remove post-merge conflict code duplication.
Thanks @seanlip & others. I addressed the conflicts & will enable auto-merge. |
Explanation
This updates Bazel code ownership to ensure that all Bazel files are reviewed by the same team members regardless of local directory ownership. This is becoming more important now that modules are being split up, and because the team isn't yet fully ramped up on maintaining BUILD files.
Note that BUILD file code ownership is a temporary situation. Once the migration from Gradle is complete & we have comprehensive Bazel documentation, we'll be expecting directory-level code owners to take over BUILD file ownership.
This was discovered when reviewing #2959: #2959 (review).