[gentoo-proxy-maint] Developers, please try to finish what you've started reviewing

[Michał Górny]

Hi, everyone.

TL;DR: If you start reviewing some package, please reply to feedback
and continue reviewing till it's merged.


I've noticed that many pull requests get stuck after first review round.
Many of the posters address the review comments in 1-2 days and then
wait very long time for the next review. This makes it really hard to
get things done, as increasing delays often render PRs outdated,
and other developers may skip pull request with explicit 'changes
requested' note.

So, developers: please try to be more focused on doing PR work
from beginning to end. Don't review broadly. Choose a few PRs, comment
on them and wait for feedback. Preferably assign yourself whenever
you're doing something, so that we know to skip it.

If the user responds to feedback and/or updates commits, please make
sure to react appropriately. Review again. If he skipped something
without explanation, ask him about it. *If you can't find time to review
again, please discard your previous review [can you do that?], so that
others see that they need to step in.*

While ideally we would have 2 or 3 pairs of eyes on every pull request,
right now we can't afford that. Instead, getting one focused pair
of eyes that establishes direct 1:1 relation with the proxied
maintainer, and processes his pull requests with delays <24h between
each review round would be really helpful.

Thanks for all your work.

-- 
Best regards,
Michał Górny