<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jul 7, 2016 at 8:56 AM, David E DeMarle <span dir="ltr"><<a href="mailto:dave.demarle@kitware.com" target="_blank">dave.demarle@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>As Robert just pointed out, the +1 state in the review process is nothing but trouble. In practice, it seems to help us avoid responsibility and delays merging, contributing to the accumulation of stale merge requests and newbie frustration.<br></div><div data-smartmail="gmail_signature"><br></div><div data-smartmail="gmail_signature">How about we say "+1 means I approve this code and assert that it is ready to merge". Merge requests from a authorized developers should be promptly merged after the +1 by the original author. Merge requests from unauthorized developers should be merged more or less immediately by the authorized reviewer after the +1.<br></div></div></blockquote><div><br></div><div>+1 </div></div></div></div>