-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Adding a note about the Github teams to the repository management docs #15595
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.
Good to have something about the teams in the doc. However, based on the description, its appears that the Gutenberg
team is just a distinction type team (i.e. you've meaningfully contributed to 2/3 of the project here's a badge). I'm assuming that's not the case, so could the purpose of the Gutenberg
team be expanded a bit?
Also, I think there's potential for confusion between the two teams, should the names given the teams be more distinctive?
I'm not great at writing docs :). The gutenberg team is also about having access to the repository (labeling, creating branches, avoid working on forks. I think it's too late to change branches and we also can't avoid "gutenberg" in the team names as the teams are global to the WordPress org. If anyone is willing to give this a stab and clarify more, please do. |
I think the idea is 2 or 3 PRs merged with the help of the contributor to be promoted to member :) Gutenberg Core team is a subset of Gutenberg team which gives you some additional capabilities. One of them is that you get notified when someone uses |
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.
Two small nits but overall nice to have this clarification!
Co-Authored-By: Chris Van Patten <[email protected]>
Co-Authored-By: Chris Van Patten <[email protected]>
Co-Authored-By: Andrew Duthie <[email protected]>
Co-Authored-By: Andrew Duthie <[email protected]>
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, Riad! Thanks for writing this up.
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 for now. Let's get this merged.
Everything is in the title.