DOC: Update triage team nomination instructions#31089
DOC: Update triage team nomination instructions#31089melissawm wants to merge 6 commits intomatplotlib:mainfrom
Conversation
| 1. Correctly triage 2-3 issues or review 2-3 pull requests. Actually labeling or | ||
| closing issues and PRs require triage permissions, but you can verify with | ||
| comments if issues have enough information or are still valid, and suggest | ||
| changes to PRs, particularly if you can help contributors solve CI check | ||
| failures. |
There was a problem hiding this comment.
There's a lot of information here, so wonder if this would work better by being integrated into our how to triage instructions, with a link to those instructions?
|
Looks like the failure is a separate issue: |
|
I decided to review the entire doc and try a new organization, grouping together the issue-related instructions and the PR instructions. Also added notes about the "good first issue" pulling from the language recently added to the contributing guide, and added some links to the labels on github for convenience. |
|
I am trying to figure out how to add an external reference label that uses a colon in rst 🙃 |
f542002 to
6775b2d
Compare
| the triage team by nominating them through the private "Triage team nominations" | ||
| category on `Discourse <https://discourse.matplotlib.org>`__ (Note that only |
There was a problem hiding this comment.
What's the confirmation (this nomination was approved) step?/next part of the process, or is that a follow up PR?
There was a problem hiding this comment.
I think this should be a Steering Council action but I wonder if we should add a timeline for a response?
|
@melissawm should I merge or wait on "next steps" discussion? |
|
Let's discuss in the meeting re: next steps. Thanks! |
| category on `Discourse <https://discourse.matplotlib.org>`__ (Note that only | ||
| ``@maintainers`` and ``@triage`` members can see this category). The nomination | ||
| will then be confirmed by the Steering Council and the user, if accepted, will | ||
| be added to the triage team on GitHub. |
There was a problem hiding this comment.
| category on `Discourse <https://discourse.matplotlib.org>`__ (Note that only | |
| ``@maintainers`` and ``@triage`` members can see this category). The nomination | |
| will then be confirmed by the Steering Council and the user, if accepted, will | |
| be added to the triage team on GitHub. | |
| category on `Discourse <https://discourse.matplotlib.org>`__ (Note that only | |
| ``@maintainers`` and ``@triage`` members can see this category). | |
| If no objections are raised in the nomination topic after one week, a member with the `owner` role on GitHub will send an invitation email to the nominee following a template. Once the nominee responds affirmatively, they will add the nominee to the Triage group on GitHub, and close the Discourse thread with a confirmation that the nomination was accepted. |
PR summary
Updates triage team nomination instructions section with Discourse link and some clarification on what potential candidates can do before they are nominated.
PR checklist