<div dir="ltr"><div><div><div>Clint<br><br></div>This rule is not currently enabled in Cinder. This review fixes up all cases and enables it, which is absolutely 100% the right thing to do if we decide to implement this rule.<br><br></div>The purpose of this thread is to understand the value of the rule. We should either enforce it, or else explicitly decide to ignore it, and educate reviewers who manually comment on it.<br><br></div>I lean against the rule, but there are certainly enough comments coming in that I'll look and think again, which is a good result for the thread.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 25 February 2015 at 22:46, Clint Byrum <span dir="ltr"><<a href="mailto:clint@fewbar.com" target="_blank">clint@fewbar.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Excerpts from Duncan Thomas's message of 2015-02-25 10:51:00 -0800:<br>
<span class="">> Hi<br>
><br>
> So a review [1] was recently submitted to cinder to fix up all of the H302<br>
> violations, and turn on the automated check for them. This is certainly a<br>
> reasonable suggestion given the number of manual reviews that -1 for this<br>
> issue, however I'm far from convinced it actually makes the code more<br>
> readable,<br>
><br>
> Is there anybody who'd like to step forward in defence of this rule and<br>
> explain why it is an improvement? I don't discount for a moment the<br>
> possibility I'm missing something, and welcome the education in that case<br>
<br>
</span>I think we've had this conclusion a few times before, but let me<br>
resurrect it:<br>
<br>
The reason we have hacking and flake8 and pep8 and etc. etc. is so that<br>
code reviews don't descend into nit picking and style spraying.<br>
<br>
I'd personally have a private conversation with anyone who mentioned<br>
this, or any other rule that is in hacking/etc., in a review. I want to<br>
know why people think it is a good idea to bombard users with rules that<br>
are already called out explicitly in automation.<br>
<br>
Let the robots do their job, and they will let you do yours (until the<br>
singularity, at which point your job will be hiding from the robots).<br>
<div class="HOEnZb"><div class="h5"><br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">Duncan Thomas</div>
</div>