<p dir="ltr">> So should we add what we don't want<br>
to see people -1 for?</p>
<p dir="ltr">>[1] <a href="http://docs.openstack.org/infra/manual/developers.html#peer-review">http://docs.openstack.org/infra/manual/developers.html#peer-review</a></p>
<p dir="ltr">This seems right way.. but concern is do everyone follow all docs?</p>
<p dir="ltr">But atleast we should document it somewhere.</p>
<p dir="ltr">Regards,<br>
Sheel Rana</p>
<div class="gmail_quote">On Apr 11, 2016 6:52 AM, "Masayuki Igawa" <<a href="mailto:masayuki.igawa@gmail.com">masayuki.igawa@gmail.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="quoted-text">2016-04-11 9:46 GMT+09:00 Matt Riedemann <<a href="mailto:mriedem@linux.vnet.ibm.com">mriedem@linux.vnet.ibm.com</a>>:<br>
><br>
><br>
> On 4/10/2016 6:37 PM, Clint Byrum wrote:<br>
>><br>
>> Excerpts from Matt Riedemann's message of 2016-04-09 06:42:54 -0700:<br>
>>><br>
>>> There is also disincentive in +1ing a change that you don't understand<br>
>>> and is wrong and then a core comes along and -1s it (you get dinged for<br>
>>> the disagreement). And there is disincentive in -1ing a change for the<br>
>>> wrong reasons (silly nits or asking questions for understanding). I ask<br>
>>> a lot of questions in a lot of changes and I don't vote on those because<br>
>>> it would be inappropriate.<br>
>>><br>
>><br>
>> Why is disagreement a negative thing? IMO, reviewers who agree too much<br>
>> are just part of the echo chamber.<br>
>><br>
>> __________________________________________________________________________<br>
>> OpenStack Development Mailing List (not for usage questions)<br>
>> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
>> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
>><br>
><br>
> I'm not saying disagreement is a negative thing, I was saying there are<br>
> times when I've seen people -1 for crazy nits, e.g. there should be a blank<br>
> line between the bug ref and change-id in the commit message, or for asking<br>
> questions for understanding (which, btw, I'm fine with -1 for 'add a comment<br>
> because this is complicated and I didn't get it at first'). And I'm also not<br>
> crazy about piling on or agreeing with everything either. My point is I<br>
> think it's appropriate in a lot of cases to just not vote but still comment.<br>
<br>
</div>I think we have some/many implicit rules for our review. There's a<br>
document[1] for review<br>
but it doesn't mention crazy nits. So should we add what we don't want<br>
to see people -1 for?<br>
<br>
[1] <a href="http://docs.openstack.org/infra/manual/developers.html#peer-review" rel="noreferrer" target="_blank">http://docs.openstack.org/infra/manual/developers.html#peer-review</a><br>
<div class="quoted-text"><br>
><br>
> --<br>
><br>
> Thanks,<br>
><br>
> Matt Riedemann<br>
><br>
><br>
><br>
> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br>
</div><div class="elided-text">__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></blockquote></div>