<div dir="ltr"><div><div><div>Hi<br><br></div>My scale was shifted by +1 actually.<br></div>What i'm using is the following :<br><br>  +1 or 👍 (:+1:) means "I consider the changed ready for integration"<br>  +2 means "I consider the changed ready for integration and have locally tested it and verified it works."<br><br></div>I'm happy with anything, but I always apreciate when a reviewer go ahead and try out the feature during the review.<br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Mathieu Westphal<br></div></div></div>
<br><div class="gmail_quote">On Tue, Oct 3, 2017 at 4:29 PM, David E DeMarle <span dir="ltr"><<a href="mailto:dave.demarle@kitware.com" target="_blank">dave.demarle@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 dir="ltr">Our current rule is:<div><div><a href="https://gitlab.kitware.com/vtk/vtk/blob/master/Documentation/dev/git/develop.md#leading-line" target="_blank">https://gitlab.kitware.com/<wbr>vtk/vtk/blob/master/<wbr>Documentation/dev/git/develop.<wbr>md#leading-line</a></div><div><br></div><div>+1 has alway been problematic.</div><div>I've never seen a +3 comment in the wild.<br><div><br></div><div>What say we simplify to just +1 or -1 with words for everything else we need to say?</div><div><br></div><div>Any +1's out there for this suggestion?</div><div>--</div><div><div><div class="m_380801273850669410gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>David E DeMarle<br>Kitware, Inc.<br>Principal Engineer<br>21 Corporate Drive<br>Clifton Park, NY 12065-8662<br>Phone: <a href="tel:(518)%20881-4909" value="+15188814909" target="_blank">518-881-4909</a></div></div></div></div></div></div>
</div></div></div></div>
<br>______________________________<wbr>_________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/<wbr>opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=<wbr>vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">http://public.kitware.com/<wbr>mailman/listinfo/vtk-<wbr>developers</a><br>
<br>
<br></blockquote></div><br></div>