-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
MAINTAINERS.md: Add @andrik & @Aracki as repos collaborators #3005
Conversation
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.
Looks good :) this diff won't conflict with #2258, so we can merge it now.
@waldyrious, I assumed that #2988 (comment) and #2988 (comment) were the confirmations. 🤔 |
I accepted it. |
The invites are only sent once the contributor(s) agrees with the proposal, as @sbrl's comments in #2988 attest. What I meant was public confirmations that the invites have been accepted, as @Aracki did above. That's not a big deal, of course, but it helps us make sure that the closing of the issue only happens once all the steps are completed (the last one being the merge of this PR). |
I accepted it |
Thanks guys! |
I have added @andrik & @Aracki as repository collaborators.
Closes #2988.