GOV/DOC: add section to docs on triaging and triage team#19124
Merged
tacaswell merged 1 commit intomatplotlib:masterfrom Dec 23, 2020
Merged
GOV/DOC: add section to docs on triaging and triage team#19124tacaswell merged 1 commit intomatplotlib:masterfrom
tacaswell merged 1 commit intomatplotlib:masterfrom
Conversation
story645
requested changes
Dec 17, 2020
QuLogic
reviewed
Dec 17, 2020
3c5d156 to
55904c0
Compare
55904c0 to
cca74a8
Compare
QuLogic
reviewed
Dec 17, 2020
story645
approved these changes
Dec 18, 2020
Member
|
@dopplershift @timhoffm @efiring This is on the agenda for today, but if you guys are OK with adding a "triage" layer to the Membership then you can hit approve. Thanks! |
dopplershift
approved these changes
Dec 22, 2020
efiring
approved these changes
Dec 22, 2020
Member
efiring
left a comment
There was a problem hiding this comment.
Just one typo.
Looks good to me.
doc/devel/triage.rst
Outdated
| or privately) to recommend you to the triage team . If you worked | ||
| with someone on the issue triaged, they would be a good person to | ||
| ask. | ||
| 3. Responsibly exercise you new power! |
Member
Author
|
I will merge this when docs pass CI. |
51913bd to
4f47080
Compare
Member
Author
|
Testing @matplotlib/triageteam notification. |
Co-authored-by: Elliott Sales de Andrade <[email protected]> Co-authored-by: hannah <[email protected]>
4f47080 to
ff65cdd
Compare
3 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Summary
This adds documentation to add a "triage team" to our organization as discussed on today's call. The purpose of this is to:
This text is based on https://scikit-learn.org/dev/developers/bug_triaging.html and modified to better fit Matplotlib.
Task that need to be done immediately outside of a PR if we merge this:
If this catches on we may want to:
On the call I suggested 1 triage would be the threshold to be invited, but ended up with 2-3 just so that we have a bit more context to evaluate. Being optomistic, I expect most of these requests / nominations to be fulfilled.
sklearn has a slightly higher bar for inviting people to the triage team (https://scikit-learn.org/dev/governance.html#triage-team) and more ceremony, but I would rather have this be broader and less formal.
We can deal with bad-faith actors if they come up and I expect a vast majority of the issues we will have will be miss-labeled or miss-milestoned issues which are almost trivially fixable.