I don't recall doiing a +2, but I'll be careful in the future...<br><br><br><div class="gmail_quote">On Tue, Sep 11, 2012 at 3:47 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="HOEnZb"><div class="h5">On Tue, Sep 11, 2012 at 2:57 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br>

> Folks,<br>
><br>
> I recently accidentally submitted a gerrit patch. This was in spite of the<br>
> fact that I had given a -1 to the topic.<br>
><br>
> When you review a topic (not just a single change) you are presented with<br>
> two buttons: "Publish Comments" and "Publish and Submit". I must have<br>
> clicked the latter.<br>
><br>
> I suggest we remove that second button to prevent accidental merges.<br>
><br>
</div></div>Hi Bill,<br>
<br>
I think the button is useful. I just verified that even if I<br>
accidentally press the "Publish and Submit" button it will not merge<br>
it unless I set the review to +2 for the topic. This means you need to<br>
accidentally set review to +2 and hit the publish and submit button. I<br>
like the convenience of being able to review and merge a topic in one<br>
commit.<br>
<br>
The reviews on the individual commits in a topic are effectively<br>
there to aid the review, when using topic reviews the top-level topic<br>
score is the only one that matters. I would like to enhance that so<br>
that a -2 review of a commit would block it. Right now we don't have<br>
many (or any) cycles available to patch Gerrit, and would like to<br>
rebase to the latest upstream before making any additional changes.<br>
<br>
I am updating some of the wiki pages, and can add a warning about this<br>
but it doesn't look too dangerous to me as you must review a commit as<br>
good to merge and then accidentally hit the wrong button, at which<br>
stage you had already stated you felt the commit was good to merge.<br>
<span class="HOEnZb"><font color="#888888"><br>
Marcus<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>Unpaid intern in BillsBasement at noware dot com<br><br>