<div>+1. I think this is a good comment for all reviews.  I have been frustrated lately with a number of reviews that I spent time upon but didn't feel should be scored.  I think that 0 with comments should be counted as well.</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jun 26, 2015 at 2:27 PM Tim Bell <<a href="mailto:Tim.Bell@cern.ch">Tim.Bell@cern.ch</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> -----Original Message-----<br>
> From: Jeremy Stanley [mailto:<a href="mailto:fungi@yuggoth.org" target="_blank">fungi@yuggoth.org</a>]<br>
> Sent: 26 June 2015 16:42<br>
> To: OpenStack Development Mailing List (not for usage questions)<br>
> Subject: Re: [openstack-dev] [Nova] The unbearable lightness of specs<br>
><br>
> On 2015-06-25 16:39:56 +0000 (+0000), Tim Bell wrote:<br>
> [...]<br>
> > One of the problems that I’ve seen is with specs etiquette where<br>
> > people -1 because they have a question. This is a question of<br>
> > education rather than a fundamental issue with the process.<br>
><br>
> <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>
> has been updated with a 7th entry addressing this in particular.<br>
> Hopefully that will help realign reviewers on acceptable vs.<br>
> unacceptable use of -1 for certain types of questions over time.<br>
<br>
I also feel that stackalytics should credit people of a 0 review comment on specs. Currently, I think that only non-zero reviews are considered as a contribution. My understanding of the workflow is that a 0 is in many cases is the constructive way to respond and therefore should be considered as a contribution.<br>
<br>
Tim<br>
<br>
> --<br>
> Jeremy Stanley<br>
><br>
> __________________________________________________________<br>
> ________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: OpenStack-dev-<br>
> <a href="http://request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">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>
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>
</blockquote></div>