FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

GOV/DOC: add section to docs on triaging and triage team by tacaswell · Pull Request #19124 · matplotlib/matplotlib · GitHub

Repository navigation

GOV/DOC: add section to docs on triaging and triage team - #19124

Merged
tacaswell merged 1 commit into
matplotlib:masterfrom
tacaswell:doc_triage_team
Dec 23, 2020
Merged

tacaswell merged 1 commit into
matplotlib:masterfrom
tacaswell:doc_triage_team

Conversation

Copy link
Copy Markdown
Member

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:

  • provide a low barrier to formally recognize users as contributing to the maintenance of Matplotlib (by giving them power and responsibility)
  • empower more people to help where they are willing and able
  • to incentive more people to triage issues
  • (secondary) provide a ramp / pipeline to contributing PRs (and reviewing PRs!) for new contributors

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:

  • create triage_team in the github ACL system (I think only me @dopplershift and @efiring can currently do that?) with triage rights on matplotlib/matplotlib

If this catches on we may want to:

  • add more formal method of requesting to join (ex a form), but for now while we figure out if this is even a good idea, lets keep it informal.
  • add "triage team captain" or similar to the named roles in the governance docs, but again lets see if this catches on / is a good idea before we do that.

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.

tacaswell added Documentation governance needs approval from @steering-council labels Dec 16, 2020
tacaswell added this to the v3.4.0 milestone Dec 16, 2020
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated
Comment thread doc/devel/triage.rst Outdated

jklymak commented Dec 22, 2020

Copy link
Copy Markdown
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!

efiring left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Just one typo.
Looks good to me.

Comment thread 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!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

you -> your

Copy link
Copy Markdown
Member Author

I will merge this when docs pass CI.

Copy link
Copy Markdown
Member Author

Testing @matplotlib/triageteam notification.

Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
Co-authored-by: hannah <story645@gmail.com>
tacaswell merged commit 8808573 into matplotlib:master Dec 23, 2020
tacaswell deleted the doc_triage_team branch December 23, 2020 05:19
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

Labels

Documentation governance needs approval from @steering-council

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL