Thanks Simon. There are two parts to this:
* the team, with the Triage and Review permission. * Whether they can approve Mariusz’ PRs over the weekend. (And other times…) I could take the previous discussion for a Yes on the first part, and just invite more people. (I should probably just do that.) BUT I’d like to resolve the second point here (it’s needed for the examples given). I think it might need an edit to DEP 10… as per here: https://github.com/django/deps/blob/master/accepted/0010-new-governance.rst#id26 <https://github.com/django/deps/blob/master/accepted/0010-new-governance.rst#id26> If so, I’m happy to type that up but I guess some pointer from the new TB that it would be acceptable would be good. Kind Regards, Carlton > On 5 Oct 2020, at 15:46, charettes <charett...@gmail.com> wrote: > > I like the idea but I don't feel comfortable vouching for it given my > conflict of interest. > > I'll just say that It does feel weird that folks on this list which are the > main active contributors outside of a few members of the currently elected TB > don't have access to basic triaging and reviewing features on Github. It > feels like these permissions would be way more useful to them, and the active > contributors members of the TB that are not mentioned here, than to members > of the TB themselves given the rare times they have to step in on a Github > issue (mainly DEPs? so far). > > Cheers, > Simon > > Le samedi 3 octobre 2020 à 03:53:46 UTC-4, carlton...@gmail.com > <http://gmail.com/> a écrit : > Hi all. > > With the new Technical Board elected (Well done to everyone there!) we have > an important workflow that’s not covered by DEP 10. > > Specifically, this scenario: > > - An issue comes in covering some esoteric corner of the ORM — generation of > column aliases for group by clauses say. > > - Mariusz picks this up and opens a PR fixing said issue. > > - Simon reviews PR and approves it. > > - Mariusz merges. > > Up until now this has been covered by Simon’s role on the TB as, under DEP > 10, TB members are allowed to approve a Merger’s PR. Officially the PR here > would now require further approval from Claude or myself before Mariusz could > proceed. > > This is a symptom of a wider issue. > > Frequently, for example, Marisuz makes a small PR on the weekend. I'm not > working Monday, so it has to wait until Tuesday morning for me to see it and > Approve it. > > Meanwhile one or more regular contributors have taken the time to look at the > PR and approve it. > > A couple of actual examples: > > 1. Made indexes tests use required_db_features. > https://github.com/django/django/pull/13432 > <https://github.com/django/django/pull/13432> > Approved by Hasan and Tim. > > 2. Corrected docstring quotes in various code. > https://github.com/django/django/pull/13445 > <https://github.com/django/django/pull/13445> > Approved by David and Hasan . > > David, Hasan and Tim are all regular features on PRs, Trac, and here, making > PRs, reviewing, commenting, helping to triage and so on. Tim is Django's #1 > all-time contributor. Hasan has made nearly 200 commits and it there every > week taking on issues from the backlog. David is a newer arrival, has been > super active and offered good input across the board, both here and on > third-party packages. > > It seems clear that their approval should be enough for a Merger PR to > progress. > > > > > Proposal: > > Some time ago I proposed a Triage and Review Team. > > https://groups.google.com/g/django-developers/c/mUBWlG0-Jbw/m/0dtgwMwPAAAJ > <https://groups.google.com/g/django-developers/c/mUBWlG0-Jbw/m/0dtgwMwPAAAJ> > I began this but it ran into the DEP 10 changes, and I stopped working on it > to let that go through. With the new TB in place, I'd like to bring that back > now, granting PR Approval power, at least for Merge PRs. > > The idea was to have a team for folks that are active on repo. This would > provide some recognition for efforts, allow extra hands to help manage > tickets — closing spam ones seems relevant this week… 😃 — and so on. > > The GitHub Team I created has GitHub’s Triage role. This allows for one thing > to request a review from a Merger. Nick is the only member of that team > currently (see above about pausing) but it’s worked well when he's reviewed > PRs from the backlog, seen that they're ≈ready and asked for a review from > me. This has been very helpful, and quite smooth. (It’s in a similar vein to > marking tickets on Trac Ready for Commit, which also helps them to show about > the mass.) > > Ultimately I think a Triage & Review Team is an opportunity to widen the > contributor pool and spread the work. > > I would suggest the following list for members to flesh out the team > initially. Happy to add more, but the idea was currently active (if you're > offended I missed you, sorry, do shout! — not intentional. ) > > - David Smith > > - Hasan Ramezani > > - Jon Dufresne > > - Nick Pope > > - Simon Charette > > - Tim Graham > > (I didn’t include Mergers or TB members.) > > Some other points: > > - I’d like to start measuring non-commit contributions somehow. This is > something I’m thinking about but for now I think by-eye is sufficient. > > - I’d be happy to make the list twice as long. > > - As I say, currently active — we should review each cycle. > > - It’s still all men — but the team would provide a way of recognising anyone > finding Triage as a Hobby appealing, so I’d hope it would act as an incentive > to participate, with a clear path to recognition (and from there candidacy in > a TB election say…) > > Hopefully that’s clear enough. Can I ask for thoughts, and if we’re keen, > what would we need to do to make it formal? > > Thanks all. > > Kind regards, Carlton > > > > > > -- > You received this message because you are subscribed to the Google Groups > "Django developers (Contributions to Django itself)" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to django-developers+unsubscr...@googlegroups.com > <mailto:django-developers+unsubscr...@googlegroups.com>. > To view this discussion on the web visit > https://groups.google.com/d/msgid/django-developers/c34045c8-55fa-4073-b64e-20f2d719d12an%40googlegroups.com > > <https://groups.google.com/d/msgid/django-developers/c34045c8-55fa-4073-b64e-20f2d719d12an%40googlegroups.com?utm_medium=email&utm_source=footer>. -- You received this message because you are subscribed to the Google Groups "Django developers (Contributions to Django itself)" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-developers+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/4DB3A17F-7303-43AF-8D0D-7D14B5A96FE4%40gmail.com.
