<html><head><style>body{font-family:Helvetica,Arial;font-size:13px}</style></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">+1 across the board for this change.</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">H803 is ignored by a large number of projects after a rather extensive conversation on the ML last year (as I recall). The other two changes seem quite reasonable.</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Cheers,</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Morgan</div> <div id="bloop_sign_1403288854842071040" class="bloop_sign"><p><strong>—</strong><br><strong>Morgan Fainberg</strong></p></div> <div style="color:black"><br>From: <span style="color:black">Sean Dague</span> <a href="mailto:sean@dague.net">sean@dague.net</a><br>Reply: <span style="color:black">OpenStack Development Mailing List (not for usage questions)</span> <a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a><br>Date: <span style="color:black">June 20, 2014 at 11:09:41</span><br>To: <span style="color:black">OpenStack Development Mailing List (not for usage questions)</span> <a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a><br>Subject: <span style="color:black"> [openstack-dev] [hacking] rules for removal <br></span></div><br> <blockquote type="cite" class="clean_bq"><span><div><div></div><div>After seeing a bunch of code changes to enforce new hacking rules, I'd

<br>like to propose dropping some of the rules we have. The overall patch

<br>series is here -

<br>https://review.openstack.org/#/q/status:open+project:openstack-dev/hacking+branch:master+topic:be_less_silly,n,z

<br>

<br>H402 - 1 line doc strings should end in punctuation. The real statement

<br>is this should be a summary sentence. A sentence is not just a set of

words that end in a period. Squirel fast bob. It's something deeper.

<br>This rule thus isn't really semantically useful, especially when you are

<br>talking about at 69 character maximum (79 - 4 space indent - 6 quote

characters).

<br>

<br>H803 - First line of a commit message must *not* end in a period. This

<br>was mostly a response to an unreasonable core reviewer that was -1ing

<br>people for not having periods. I think any core reviewer that -1s for

<br>this either way should be thrown off the island, or at least made fun

<br>of, a lot. Again, the clarity of a commit message is not made or lost by

the lack or existence of a period at the end of the first line.

<br>

<br>H305 - Enforcement of libraries fitting correctly into stdlib, 3rdparty,

<br>our tree. This biggest issue here is it's built in a world where there

<br>was only 1 viable python version, 2.7. Python's stdlib is actually

<br>pretty dynamic and grows over time. As we embrace more python 3, and as

<br>distros start to make python3 be front and center, what does this even

<br>mean? The current enforcement can't pass on both python2 and python3 at

the same time in many cases because of that.

<br>

We have to remember we're all humans, and it's ok to have grey space.

<br>Like in 305, you *should* group the libraries if you can, but stuff like

<br>that should be labeled as 'nit' in the review, and only ask the author

to respin it if there are other more serious issues to be handled.

<br>

<br>Let's optimize a little more for fun, and stop throwing -1s for silly

<br>things. :)

<br>

<br>      -Sean

<br>

<br>--  

<br>Sean Dague

<br>http://dague.net

<br>

<br><hr>_______________________________________________

<br>OpenStack-dev mailing list

<br>OpenStack-dev@lists.openstack.org

<br>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

<br></div></div></span></blockquote></body></html>