*I assume we ultimately need a committer to review the work of
anon-committer ahead of merge ( since someone with permissions
eventuallyhas to actually accept/merge/commit the code ).  In general, is
onereviewer/committer sufficient?*
IMO, yes, one reviewer who is also a committer is sufficient

*How does this change if the author of the PR is an existing committer?*
If I am a committer and I make a PR, I would wait for another committer to
review and approve. However, if it's been a week and no one approves, I
would send a review request to dev@, and if there is no response after a
couple of weeks, then I'd go ahead and merge it myself (given all
tests/checks have passed of course)

Thanks for the discussion!
- Gedd

Reply via email to