You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Nominate Joannah Nanjekye to Python Triage Team #358
@willingc I assume @nanjekyejoannah is good with this nomination? Otherwise you being a core dev doing the nomination is an implicit approval/endorsement which means we just need Joannah to say she wants the responsibility. 😃
Since @nanjekyejoannah already has the bug triage permission on bugs.python.org, I'm fine with adding her to the GitHub triage team.
But As I wrote for @corona10's nomination #357 (comment), I would prefer to provide a mentor to @nanjekyejoannah to help her to handle her new responsibilities (ex: close a PR).
Since I'm already mentoring @nanjekyejoannah, I would be ok to mentor her for her new responsibilities.
Reacted by Joannah Nanjekye, Carol Willing and Donghee Na
FYI I sent the following email to @nanjekyejoannah. I asked her to double check with me before using new "dangerous" operations (more dangerous than closing a bug at bugs.python.org).
Hi Joannah,
Congrats, you got the bug triage permission on the Python organization of GitHub! #358
Compared to your previous bug triage role on bugs.python.org, you can now also close/re-open a pull request on GitHub (and close/re-open bugs on Python projects other than cpython, cpython still uses bugs.python.org for bugs). You can also apply labels.
During one month, I would prefer that you ask me before closing a pull request or before adding labels (especially "automerge" and "skip issue") to a pull request. Just to double check ;-)
FYI I sent the following email to @nanjekyejoannah. I asked her to double check with me before using new "dangerous" operations (more dangerous than closing a bug at bugs.python.org).
Hi Joannah,
Congrats, you got the bug triage permission on the Python organization of GitHub! #358
Compared to your previous bug triage role on bugs.python.org, you can now also close/re-open a pull request on GitHub (and close/re-open bugs on Python projects other than cpython, cpython still uses bugs.python.org for bugs). You can also apply labels.
During one month, I would prefer that you ask me before closing a pull request or before adding labels (especially "automerge" and "skip issue") to a pull request. Just to double-check ;-)
Victor
Noted and Thank you @vstinner. FYI, the email might have been spammed :). I will double-check.
During one month, I would prefer that you ask me before closing a pull request or before adding labels (especially "automerge" and "skip issue") to a pull request. Just to double-check ;-)
Triagers can't really use the automerge label, it's automatically removed by the bots if anyone other than a core dev attempts to add it (according to Mariatta). I was initially concerned about this since triagers weren't supposed to be able to directly or indirectly merge PRs.
As far as skip issue goes, I've only been using it myself for PRs that involve fairly trivial documentation changes, such as grammar/typo corrections. It might also be appropriate for manual backports.
Based on the recent feedback in python/devguide#530, it seems that we're going to have triagers suggest that GitHub PRs/issues should be closed through the usage of the new stale label and the already existing invalid label, but have the core devs responsible for the actual closing.
Activity
corona10 commentedon Aug 26, 2019
Awesome!
brettcannon commentedon Aug 26, 2019
@willingc I assume @nanjekyejoannah is good with this nomination? Otherwise you being a core dev doing the nomination is an implicit approval/endorsement which means we just need Joannah to say she wants the responsibility. 😃
nanjekyejoannah commentedon Aug 26, 2019
Thanks, @willingc for nomination. @brettcannon and others, I consent to responsibilities.
willingc commentedon Aug 27, 2019
@brettcannon @Mariatta Would a checkbox on the issue template be helpful to state that the person consents when proposed by a core dev?
vstinner commentedon Aug 27, 2019
+1 for me as well
Since @nanjekyejoannah already has the bug triage permission on bugs.python.org, I'm fine with adding her to the GitHub triage team.
But As I wrote for @corona10's nomination #357 (comment), I would prefer to provide a mentor to @nanjekyejoannah to help her to handle her new responsibilities (ex: close a PR).
Since I'm already mentoring @nanjekyejoannah, I would be ok to mentor her for her new responsibilities.
Mariatta commentedon Aug 27, 2019
Sorry, but I need a break from open source and about to be OOOS, so I'm unassigning myself. Can someone else look into this?
brettcannon commentedon Aug 27, 2019
I'm looking into it.
ewdurbin commentedon Aug 27, 2019
Invite sent.
vstinner commentedon Aug 29, 2019
FYI I sent the following email to @nanjekyejoannah. I asked her to double check with me before using new "dangerous" operations (more dangerous than closing a bug at bugs.python.org).
Hi Joannah,
Congrats, you got the bug triage permission on the Python organization of GitHub!
#358
Compared to your previous bug triage role on bugs.python.org, you can now also close/re-open a pull request on GitHub (and close/re-open bugs on Python projects other than cpython, cpython still uses bugs.python.org for bugs). You can also apply labels.
During one month, I would prefer that you ask me before closing a pull request or before adding labels (especially "automerge" and "skip issue") to a pull request. Just to double check ;-)
Victor
willingc commentedon Aug 29, 2019
Thanks @vstinner.
nanjekyejoannah commentedon Aug 29, 2019
Noted and Thank you @vstinner. FYI, the email might have been spammed :). I will double-check.
vstinner commentedon Aug 29, 2019
Oh. I sent it from my new email address: Victor Stinner vstinner@python.org.
aeros commentedon Aug 29, 2019
Congrats @nanjekyejoannah!
@vstinner:
Triagers can't really use the
automergelabel, it's automatically removed by the bots if anyone other than a core dev attempts to add it (according to Mariatta). I was initially concerned about this since triagers weren't supposed to be able to directly or indirectly merge PRs.As far as
skip issuegoes, I've only been using it myself for PRs that involve fairly trivial documentation changes, such as grammar/typo corrections. It might also be appropriate for manual backports.Based on the recent feedback in python/devguide#530, it seems that we're going to have triagers suggest that GitHub PRs/issues should be closed through the usage of the new
stalelabel and the already existinginvalidlabel, but have the core devs responsible for the actual closing.vstinner commentedon Aug 30, 2019
It's fine for grammar/typo corrections. For backports, the bot which creates backport PRs automatically copies labels.