<div dir="ltr">Renat,<div><br></div><div>Yes, I have created blueprint, here is a link [1] . </div><div>In this bp you can find link to the etherpad where I described a couple of rules that came to my mind.</div><div><br></div><div><br></div><div>[1] <a href="https://blueprints.launchpad.net/mistral/+spec/add-custom-code-style-checks">https://blueprints.launchpad.net/mistral/+spec/add-custom-code-style-checks</a></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 24, 2015 at 10:59 AM, Renat Akhmerov <span dir="ltr"><<a href="mailto:rakhmerov@mirantis.com" target="_blank">rakhmerov@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 style="word-wrap:break-word">I’m the one who has been enforcing those style things from the beginning so let me take this and describe these rules in details.<div><br></div><div>Anastasia, have you already created a BP?</div><div><span class="HOEnZb"><font color="#888888"><br><div>
<div>Renat Akhmerov</div><div>@ Mirantis Inc.</div><div><br></div><br>
</div></font></span><div><div class="h5">
<br><div><blockquote type="cite"><div>On 10 Dec 2015, at 20:05, Anastasia Kuznetsova <<a href="mailto:akuznetsova@mirantis.com" target="_blank">akuznetsova@mirantis.com</a>> wrote:</div><br><div><div dir="ltr">Moshe,<div><br></div><div>I will create blueprint for that and will attach link to etherpad, so we can form list of the rules all together. </div><div>After that it will be possible to publish all our 'rules' to docs and start their implementation.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 10, 2015 at 11:23 AM, ELISHA, Moshe (Moshe) <span dir="ltr"><<a href="mailto:moshe.elisha@alcatel-lucent.com" target="_blank">moshe.elisha@alcatel-lucent.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div lang="EN-US" link="blue" vlink="purple">
<div><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Thanks, Anastasia!<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Who can take start documenting the rules? I remember only a few rules and I don’t know all the nuances.<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">For example, if the return statement is the only statement of a function – do you still need a blank line before it?<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Once the rules doc will be available I can work on adding these rules to our pep8.<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> Anastasia Kuznetsova [mailto:<a href="mailto:akuznetsova@mirantis.com" target="_blank">akuznetsova@mirantis.com</a>]
<br>
<b>Sent:</b> Wednesday, December 09, 2015 1:13 PM<br>
<b>To:</b> OpenStack Development Mailing List (not for usage questions)<br>
<b>Subject:</b> Re: [openstack-dev] [mistral] Improving Mistral pep8 rules files to match Mistral guidelines<u></u><u></u></span></p><div><div><p class="MsoNormal"><u></u> <u></u></p>
<div><p class="MsoNormal">Hi Moshe,<u></u><u></u></p>
<div><p class="MsoNormal"><u></u> <u></u></p>
</div>
<div><p class="MsoNormal">Great idea! <u></u><u></u></p>
</div>
<div><p class="MsoNormal"><u></u> <u></u></p>
</div>
<div><p class="MsoNormal">It is possible to prepare some additional code checks, for example you can take a look how it was done in Rally project [1]. <br>
Before starting such work in Mistral, I guess that we can describe our addition code style rules in our official docs (somewhere in "Developer Guide" section [2]).<u></u><u></u></p>
</div>
<div><p class="MsoNormal"><u></u> <u></u></p>
</div>
<div><p class="MsoNormal">[1] <a href="https://github.com/openstack/rally/tree/master/tests/hacking" target="_blank">https://github.com/openstack/rally/tree/master/tests/hacking</a><u></u><u></u></p>
</div>
<div><p class="MsoNormal">[2] <a href="http://docs.openstack.org/developer/mistral/#developer-guide" target="_blank">http://docs.openstack.org/developer/mistral/#developer-guide</a><u></u><u></u></p>
</div>
</div>
<div><p class="MsoNormal"><u></u> <u></u></p>
<div><p class="MsoNormal">On Wed, Dec 9, 2015 at 11:21 AM, ELISHA, Moshe (Moshe) <<a href="mailto:moshe.elisha@alcatel-lucent.com" target="_blank">moshe.elisha@alcatel-lucent.com</a>> wrote:<u></u><u></u></p>
<div>
<div><p class="MsoNormal">Hi all,<u></u><u></u></p><p class="MsoNormal"> <u></u><u></u></p><p class="MsoNormal">Is it possible to add all / some of the special guidelines of Mistral (like blank line before return, period at end of comment, …) to our pep8 rules file?<u></u><u></u></p><p class="MsoNormal"> <u></u><u></u></p><p class="MsoNormal">This can save a lot of time for both committers and reviewers.<u></u><u></u></p><p class="MsoNormal"> <u></u><u></u></p><p class="MsoNormal">Thanks!<u></u><u></u></p>
</div>
</div><p class="MsoNormal" style="margin-bottom:12.0pt"><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><u></u><u></u></p>
</div><p class="MsoNormal"><br>
<br clear="all">
<u></u><u></u></p>
<div><p class="MsoNormal"><u></u> <u></u></p>
</div><p class="MsoNormal">-- <u></u><u></u></p>
<div>
<div><p class="MsoNormal">Best regards,<u></u><u></u></p>
<div><p class="MsoNormal">Anastasia Kuznetsova<u></u><u></u></p>
</div>
</div>
</div>
</div>
</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" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr">Best regards,<div>Anastasia Kuznetsova</div></div></div>
</div>
__________________________________________________________________________<br>OpenStack Development Mailing List (not for usage questions)<br>Unsubscribe: <a href="mailto:OpenStack-dev-request@lists.openstack.org" target="_blank">OpenStack-dev-request@lists.openstack.org</a>?subject:unsubscribe<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></blockquote></div><br></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" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Best regards,<div>Anastasia Kuznetsova</div></div></div>
</div>