<div dir="ltr"><div>"+1 and +2" are familiar to all of us since we learned them all (after a while) from gerrit. We should continue to strive to make it easy on newbies too.</div><div><br></div><div>+1 for buttons</div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature">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>
<br><div class="gmail_quote">On Wed, Apr 15, 2015 at 4:20 PM, Bill Lorensen <span dir="ltr"><<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Buttons are great. No need to remember the quirks of the process.<br>
<span class="im HOEnZb"><br>
<br>
On Wed, Apr 15, 2015 at 4:15 PM, Sean McBride <<a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a>> wrote:<br>
> Even that is hard to remember (though better). Would it not be possible to put buttons with names like "Approved" and "Approved & Tested". Then one has nothing to remember.<br>
><br>
> Sean<br>
><br>
><br>
><br>
> On Wed, 15 Apr 2015 15:48:43 -0400, Robert Maynard said:<br>
><br>
>>Hi Bill,<br>
>><br>
>>I was discussing this with other developers a few days ago and<br>
>>mentioned that the Acked-by/etc are hard to remember and proposed that<br>
>>we allow +1 to mean approved by a developer, and +2 to mean approved<br>
>>and tested.<br>
>><br>
>>On Wed, Apr 15, 2015 at 3:27 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>><br>
>>wrote:<br>
>>> To approve a gitlab topic I do?<br>
>>><br>
>>> Acked-by: me<br>
>>><br>
>>> Why not?<br>
>>><br>
>>> Approved-by: me<br>
>>><br>
>>> The latter seems more descriptive.<br>
>>><br>
>>> I currently contribute to 3 open source projects, vtk(gitlab),<br>
>>> itk(gerrit) and slicer(github). They each have their own workflow. I<br>
>>> also have several of my own projects that uses github. I contribute to<br>
>>> these projects in spurts.<br>
><br>
><br>
<br>
<br>
<br>
</span><span class="im HOEnZb">--<br>
Unpaid intern in BillsBasement at noware dot com<br>
</span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" target="_blank">http://markmail.org/search/?q=vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" target="_blank">http://public.kitware.com/mailman/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br></div>