<div dir="ltr"><div>Hey Folks,<br><br></div><div class="gmail_signature" data-smartmail="gmail_signature">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><br></div><div class="gmail_signature" 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><br></div><div class="gmail_signature" data-smartmail="gmail_signature">In either case both authors and reviewers are responsible for watching the dashboards and addressing issues that come up afterward.<br></div><div class="gmail_signature" data-smartmail="gmail_signature"><br>David E DeMarle<br>Kitware, Inc.<br>R&D Engineer<br>21 Corporate Drive<br>Clifton Park, NY 12065-8662<br>Phone: 518-881-4909</div><div><div>
</div></div></div>