-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
CODEOWNERS: Remove the catch-all rule #32174
Conversation
9459cdb
to
3e8ff7b
Compare
3e8ff7b
to
24b638a
Compare
24b638a
to
88269d1
Compare
for the following files it might make sense to create a new
|
88269d1
to
6b536ae
Compare
/test |
Pull Request is not mergeable
Pull Request is not mergeable
6b536ae
to
2432b03
Compare
/test |
Oh, TIL https://github.com/cilium/cilium/blob/main/.github/workflows/lint-codeowners.yaml. Looks like we don't really need another codeowners checker then. Thanks Michi! |
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 aside from the releases team not being created yet.
2432b03
to
6a8aa39
Compare
6a8aa39
to
937e487
Compare
/test |
need a quick ack from @cilium/docs-structure team for the auto-generated Documentation/codeowners.rst file 🚀🙏 |
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.
Approving for codeowners. One more minor nit.
Remove the catch-all rule to avoid pulling in tophat for review whenever new files get added. Pulling in tophat is unnecessary because: - lint-codeowners.yaml [^1] validates CODEOWNERS has entries for new files. - cilium/contributing team owns CODEOWNERS file. This team can ensure that new files get placed in appropriate locations. [^1]: https://github.com/cilium/cilium/blob/main/.github/workflows/lint-codeowners.yaml Suggested-by: Timo Beckers <timo@isovalent.com> Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
937e487
to
b71f94a
Compare
/test |
Remove the catch-all rule to avoid pulling in tophat for review whenever new files get added. Pulling in tophat is unnecessary because:
Suggested-by: Timo Beckers timo@isovalent.com
Footnotes
https://github.com/cilium/cilium/blob/main/.github/workflows/lint-codeowners.yaml ↩