<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Wed, Oct 10, 2018 at 2:35 PM Greg Hill <<a href="mailto:gnhill@liquidweb.com">gnhill@liquidweb.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'm not sure how using pull requests instead of Gerrit changesets would <br>
help "core reviewers being pulled on to other projects"?<br></blockquote><div><br></div><div>The 2 +2 requirement works for larger projects with a lot of contributors. When you have only 3 regular contributors and 1 of them gets pulled on to a project and can no longer actively contribute, you have 2 developers who can +2 each other but nothing can get merged without that 3rd dev finding time to add another +2. This is what happened with Taskflow a few years back. Eventually the other 2 gave up and moved on also.</div></div></div></blockquote><div><br></div><div>As a note, the 2+2 requirement is only a convention, not a rule. Swift has moved to 1+2 already, and other projects have considered it.<br></div><div><br></div><div>// jim</div></div></div>