-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Fix CODEOWNERS #1925
Fix CODEOWNERS #1925
Conversation
This reverts commit d789044.
Thanks! |
Do we want to add a validation action here, given that this file breaks silently? Originally suggested in #1920 (comment) |
Probably, but I'm not sure how since I'm not aware of any official GitHub validation tool. |
https://github.com/mszostok/codeowners-validator seems like the best choice right now. |
I'm honestly more worried about people forgetting to add new PEPs then breaking it with future updates. The format is simple enough that people adding themselves should generally work. |
Fixes #1920
This version works, because I opened #1924 against this branch and it correctly requested review from Brett.
This makes three changes: