<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 5, 2014 at 5:28 PM, Doug Hellmann <span dir="ltr"><<a href="mailto:doug.hellmann@dreamhost.com" target="_blank">doug.hellmann@dreamhost.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


<div><br>
> The assert ones do seem to fit the best practices as I understand them, but<br>
> I suspect there's going to be quite a bit of work to get projects compliant.<br>
<br>
</div>I've seen some work being done on that already, but I don't know how<br>
strongly we care about those specific rules as an overall project.<br><span><font color="#888888"><br></font></span></blockquote></div><div class="gmail_extra"><br></div><div class="gmail_extra">I created the Keystone blueprint[1] to automate the things we already check for in reviews. My motivation was to make it faster for contributors to contribute because they would get feedback before getting a bunch of -1s in Gerrit. I also wanted to free up core dev resources so that we can focus on more important parts of reviews.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">I'd be happy to start putting some of these in hacking, but I don't know which rules would be acceptable to all projects. Maybe there is a way to make optional checks that can be enabled in the tox.ini</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">1. <a href="https://blueprints.launchpad.net/keystone/+spec/more-code-style-automation">https://blueprints.launchpad.net/keystone/+spec/more-code-style-automation</a></div>
<div><br></div>-- <br>David<br>blog: <a href="http://www.traceback.org" target="_blank">http://www.traceback.org</a><br>twitter: <a href="http://twitter.com/dstanek" target="_blank">http://twitter.com/dstanek</a><div>www: <a href="http://dstanek.com" target="_blank">http://dstanek.com</a></div>


</div></div>