We have a lot of PRs, and we’ve spawned a couple of techniques for getting them merged:
- The recent “PRs ready for review” threads.
- Users (notably @worldofpeace ) manually scanning through PRs to find things that need attention.
- People often mention random committers who’ve been helpful in the past if things get stuck.
All of these are attempts to get active committers to look at PRs and get them through the final steps to being merged. I wonder whether we could ease this by adding a
- People can be added to
pr-shepherdsif they’re committers and they’re keen to help out with other people’s PRs.
- Users can mention
pr-shepherdsif their PR is languishing (equivalent of posting in “PRs ready for review”).
- We can add a line to the issue template telling people about this and especially recommending it for newcomers (hopefully alleviates the need to manually look through lots of PRs).
- I think it should be “shepherds” rather than reviewers, since often the most useful reviews are giving by unprivileged maintainers or other users, and the shepherd just does a final check and merge.
This overlaps a bit with RFC-0039, but that’s about expanding the number of people who can do maintainer-y things, this is just about connecting existing keen maintainers with places where they can help.