<div dir="ltr">James, do you have any thoughts on it?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 14, 2014 at 12:16 AM, Sergey Lukjanov <span dir="ltr"><<a href="mailto:slukjanov@mirantis.com" target="_blank">slukjanov@mirantis.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi folks,</div><div><br></div><div>I think that we can enable hacking checks for python infra projects to help themselves write better code. Probably we should enable only specific subset of hacking checks instead of all of them.</div>
<div><br></div><div>TL;DR</div><div><br></div><div>I'd like to discuss the need of hacking usage for python infra projects. Currently I'm keeping in mind several of them as examples - Zuul and Nodepool. The question was catched up by James E. Blair when I've created some patches for Zuul [1], so, thank you, James, for catching this.</div>
<div><br></div><div>There are some pros and cons of enabling hacking for them. As a big pros I see following the common OpenStack processes and additional checks that will help us to write a better code. On the other side, these projects are not so big for having such strong checks.</div>
<div><br></div><div>My personal opinion is that it'll be better to enable hacking checks (probably not all of them) for such projects and I'm volunteering to work on it and to help coordinate efforts if needed.</div>
<div><br></div><div>BTW I've already done some part of this work for Zuul in [1], it's head of the CR chain.</div><div><br></div><div>So, please, share your thoughts.</div><div><br></div><div>Thanks.</div><div><br>
</div><div>[1] <a href="https://review.openstack.org/#/c/63921/" target="_blank">https://review.openstack.org/#/c/63921/</a></div><span class="HOEnZb"><font color="#888888"><div><div><div><div><br></div>-- <br><div dir="ltr">
<div>Sincerely yours,</div><div>Sergey Lukjanov</div><div>
Savanna Technical Lead</div><div>Mirantis Inc.</div></div>
</div></div></div></font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><div>Sincerely yours,</div><div>Sergey Lukjanov</div><div>Savanna Technical Lead</div><div>Mirantis Inc.</div></div>
</div>