<p dir="ltr">+2. I think the having checks are helpful.  If they aren't removed it does Leelee harm. </p>
<p dir="ltr">Jay</p>
<div class="gmail_quote">On Mar 15, 2015 6:39 AM, "Duncan Thomas" <<a href="mailto:duncan.thomas@gmail.com">duncan.thomas@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 13 March 2015 at 17:36, Doug Hellmann <span dir="ltr"><<a href="mailto:doug@doughellmann.com" target="_blank">doug@doughellmann.com</a>></span> wrote:<br><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I wish we, as a community, were less obsessed with creating so many<br>
hacking rules. These are really minor changes and it's going to be a<br>
relatively short-lived issue that could just be fixed once. If there's a<br>
regression, fixing *that* won't be hard or take long.<br><br></blockquote><div><br></div><div>Ok, this comment has peaked my interest again, since I hold pretty much the exact opposite view and am a well known fan of hacking checks. My logic is:</div><div>- Hacking checks point out the error for most submitters before review, saving reviewer time and CI cycles, and increasing the comfort level of the submitter (for most people, a -1 feels harsh/negative)</div><div>- Hacking checks don't slip up and miss one because they are reviewing at 1am the night before the deadline</div><div>- Regressions hurt, and not all of our code is covered by unit tests / CI</div><div>- Debugging the output of a hacking check is many times faster than debugging a unit test, even for simple failures (and work is underway to may this even faster)</div><div>- Hacking checks that are left around for migrations like this even a few cycles longer than they are needed have zero cost. As long as nobody type 'oslo.foo' then they never need even know of their existence. We can be really lazy about removing them with no harm at all.</div><div><br></div><div>Am I missing something? I am a general proponent of 'write hacking checks for any mechanical code change'. We've seen definite benefits from this approach and few to no downsides that I'm aware of.</div></div>
</div></div>
<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>
<br></blockquote></div>