Extend basic privileges of the PyPA GitHub org members to `Triage`

Hi,

Even though I have commit/admin privileges on one project under PyPA, I often find myself participating in various review activities in other repositories under the GitHub orgs.

Sometimes after some initial discussion/reviews there’s a need to request a review from the maintainers of that specific project to help contributors move their PRs forward.

One workaround would be to tag those maintainers in comments. But this may flood PR subscribers with unnecessary notifications. If I only want to notify specific folks/teams I’d like to use a review feature instead.

The current privilege setting is “Read”, I suggest switching it to “Triage” (see: https://docs.github.com/en/free-pro-team@latest/github/setting-up-and-managing-organizations-and-teams/repository-permission-levels-for-an-organization#repository-access-for-each-permission-level)

This will basically allow any PyPA org member to have a little bit more non-destructive access to other projects under the PyPA umbrella.

One of the most annoying places where I cannot request reviews is pypa/packaging.python.org which is technically a common maintainership project but only several people can perform these simple tasks there…

3 Likes

Makes sense to me. We have a “Gardeners” team already (https://github.com/orgs/pypa/teams/gardeners) but it’s not widely used. If we can just set this across all PyPA org members instead, I don’t see why not.

1 Like

I never realized what was its purpose, FWIW looks like only @sumanah is on that team.

Without thinking about whatever the PEP 609 world means, I’m on board for this too!

1 Like

Very good idea as this should help dealing with fresh issues and PRs, and lower the load on cores.

@dustin it doesn’t look like we’ll get any extra opinions here. Sounds like a PEP 609 vote needs to be initiated, right?

I don’t think this requires a vote, as it doesn’t fall under anything listed at https://www.python.org/dev/peps/pep-0609/#pypa-committer-votes. I think it just makes sense and we should just do it.

2 Likes

Alright, so I guess you could go ahead and implement this. I think this still may require announcing the change on the ML so that folks won’t be caught by surprise.

Looks like it’s not possible to set “Triage” or “Maintain” as a base permission:

Oh, is PyPA on a legacy tariff plan by any chance?

I’ve checked a few of my orgs and none of them seems to allow using Triage as a base privilege…