I hit the wrong button by accident. I did not give it a +2. I'm pretty sure I gave it (or a change) a -1.<br><br>I don't see the advantage of the Publish and Submit.<br><br><br><br><div class="gmail_quote">On Tue, Sep 11, 2012 at 5:05 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">Agreed, but at that point it is also approved for merging. If you<br>
don't want it to be merged don't give it a +2, a +1 is a good way to<br>
say I think it is good but really want someone else to view it. The<br>
submit change button also becomes available on the main topic page<br>
after a +2, which will not ask for confirmation either.<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Sep 11, 2012 at 4:53 PM, David Cole <<a href="mailto:david.cole@kitware.com">david.cole@kitware.com</a>> wrote:<br>
> Well, *anybody* with approval rights<br>
><br>
> On Tue, Sep 11, 2012 at 4:53 PM, David Cole <<a href="mailto:david.cole@kitware.com">david.cole@kitware.com</a>> wrote:<br>
>> The button is available if *anybody* gives it a +2<br>
>><br>
>><br>
>> On Tue, Sep 11, 2012 at 4:13 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br>
>>> I don't recall doiing a +2, but I'll be careful in the future...<br>
>>><br>
>>><br>
>>><br>
>>> On Tue, Sep 11, 2012 at 3:47 PM, Marcus D. Hanwell<br>
>>> <<a href="mailto:marcus.hanwell@kitware.com">marcus.hanwell@kitware.com</a>> wrote:<br>
>>>><br>
>>>> On Tue, Sep 11, 2012 at 2:57 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>><br>
>>>> wrote:<br>
>>>> > Folks,<br>
>>>> ><br>
>>>> > I recently accidentally submitted a gerrit patch. This was in spite of<br>
>>>> > 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<br>
>>>> > 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>
>>>> 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>
>>>><br>
>>>> Marcus<br>
>>><br>
>>><br>
>>><br>
>>><br>
>>> --<br>
>>> Unpaid intern in BillsBasement at noware dot com<br>
>>><br>
>>><br>
>>> _______________________________________________<br>
>>> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>><br>
>>> Visit other Kitware open-source projects at<br>
>>> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
>>><br>
>>> Follow this link to subscribe/unsubscribe:<br>
>>> <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
>>><br>
>>><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Unpaid intern in BillsBasement at noware dot com<br><br>