The rules for a merge sound complicated to me.<div><br></div><div>But, I will be very careful in the future.<br><br><div class="gmail_quote">On Tue, Sep 11, 2012 at 6:10 PM, Marcus D. Hanwell <span dir="ltr"><<a href="mailto:marcus.hanwell@kitware.com" target="_blank">marcus.hanwell@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 class="im">On Tue, Sep 11, 2012 at 5:39 PM, Sean McBride <<a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a>> wrote:<br>

> On Tue, 11 Sep 2012 17:19:12 -0400, Bill Lorensen said:<br>
><br>
>>I hit the wrong button by accident. I did not give it a +2. I'm pretty sure<br>
>>I gave it (or a change) a -1.<br>
><br>
> IIRC, the topic had several changes, and some of the changes I think were given +2.<br>
><br>
> Is it possible that the whole topic can get merged if any one of, but not all, of the sub-changes are at +2?<br>
><br>
</div>For Gerrit topic review it does not matter what any of the individual<br>
changes get as a review. The entire topic must have a Verified +1, and<br>
a Code Review +2 score in order to be merged. Reviewers can block a<br>
merge by giving a -2 at the topic level, that topic had a single +2<br>
given by one user and no other scores.<br>
<br>
Any -1 score does not block a commit, any +1 does not enable the<br>
submit and merge or submit change button. If you review a commit and<br>
it does not have a +1 Verified and +2 Code Review you will see an<br>
error page informing you the change needs Verified and Code Review. I<br>
tested this myself on a change earlier today, it blocks merging until<br>
both those scores are in.<br>
<br>
I can't see any way that an accidental merge can be done in less than<br>
two steps (+2 Code Review score, and then hitting Publish and Submit).<br>
It is possible there is a bug, but I was not able to trigger it (and I<br>
tried a few times).<br>
<span class="HOEnZb"><font color="#888888"><br>
Marcus<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>Unpaid intern in BillsBasement at noware dot com<br><br>
</div>