-
-
Notifications
You must be signed in to change notification settings - Fork 31.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
Add @maxking to CODEOWNERS file #13599
Conversation
Hmm we've never tried adding non core developers into But anyway, let's try this, and we can always revert the change. |
Thanks @Mariatta ! Now all I need to do is open a PR and see if it works ;-) |
https://help.github.com/en/articles/about-code-owners
|
This reverts commit 71dc7c5.
I opened #13660 to revert this since @tirkarthi found that reference saying @maxking would need to have write access to get notifications. Maybe once the new "triage" access is launched that will work with CODEOWNERS and then we can give @maxking that level of access. |
No description provided.