Skip to content

Conversation

@gregestren
Copy link
Collaborator

  • Fix incorrect directories.
  • Update to best-guess correct owners, preferring team labels when they exist.
  • Exclude directories with one-off contributions: it's fine to keep default repo ownership for these.

- Fix incorrect directories.
- Update to best-guess correct owners, preferring team labels when they exist.
- Exclude directories with one-off contributions: it's fine to keep default repo  ownership for these.
@gregestren gregestren requested a review from Wyverald as a code owner July 30, 2025 19:07
Copy link
Member

@Wyverald Wyverald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure if the team owners like @bazelbuild/configurability ever worked for this repo (Github seems to throw errors on them), but up to you whether you want to keep those.

@gregestren
Copy link
Collaborator Author

I noticed that too. Since that's pre-existing, I'll submit as-is then do some test PR (or better yet a real PR) to see if it works.

@gregestren
Copy link
Collaborator Author

CI failures are at head, so I'll merge this and consider the CI failures independently.

@gregestren gregestren merged commit c7db6b0 into bazelbuild:main Aug 1, 2025
1 of 2 checks passed
@gregestren gregestren deleted the codeowners branch August 1, 2025 19:55
@gregestren
Copy link
Collaborator Author

CI failure details: #610 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants