<div dir="ltr">Adding infra also, as might be relevant for our code.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 9, 2016 at 11:06 AM, Sandro Bonazzola <span dir="ltr"><<a href="mailto:sbonazzo@redhat.com" target="_blank">sbonazzo@redhat.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">Hi,<div>in project maintained by integration team we tried to adhere to some styling rules while writing python code, one of these was related to the import lines. After a lot of time doing this manually, we discovered isort ( <a href="https://pypi.python.org/pypi/isort/4.2.5" target="_blank">https://pypi.python.org/<wbr>pypi/isort/4.2.5</a> )</div><div><br></div><div>We're using it before we push our code to gerrit and a few days ago I sent a patch for the python code within ovirt-engine: <a href="https://gerrit.ovirt.org/61964" target="_blank">https://gerrit.<wbr>ovirt.org/61964</a></div><div><br></div><div>The configuration we're currently using is:</div><div><br></div><div><span style="color:rgb(53,53,53);font-family:monospace;white-space:pre-wrap">$ cat ~/.isort.cfg
[settings]
line_length=79
known_standard_library=<wbr>configparser,Cheetah.Template
force_single_line=True
default_section=FIRSTPARTY
known_otopi=otopi
known_host_deploy=ovirt_host_<wbr>deploy
known_ovirt_engine=ovirt_<wbr>engine
known_ovirt_engine_setup=<wbr>ovirt_engine_setup
known_ovirt_setup_lib=ovirt_<wbr>setup_lib
known_vdsm=vdsm
known_ovirt_hosted_engine_<wbr>setup=ovirt_hosted_engine_<wbr>setup
sections=FUTURE,STDLIB,<wbr>FIRSTPARTY,OTOPI,VDSM,HOST_<wbr>DEPLOY,OVIRT_ENGINE,OVIRT_<wbr>ENGINE_SETUP,OVIRT_SETUP_LIB,<wbr>OVIRT_HOSTED_ENGINE_SETUP,<wbr>THIRDPARTY,LOCALFOLDER
lines_between_types=2</span></div><div><font color="#353535" face="monospace"><span style="white-space:pre-wrap"><br></span></font></div>It has been proposed to add 'isort --check-only' in check-patch.for python projects in addition to pep8 style checking.<br><br>It has also been proposed to reach consensus on isort configuration so all the python code<br>within ovirt project will have same styling.<br><br>I'd like to get some feedback about these proposal in order to decide how to procede.<br><br>Thanks,<span class="HOEnZb"><font color="#888888"><div><div><br></div>-- <br><div data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr">Sandro Bonazzola<br>Better technology. Faster innovation. Powered by community collaboration.<br>See how it works at <a href="http://redhat.com" target="_blank">redhat.com</a><br></div></div></div></div>
</div></font></span></div>
<br>______________________________<wbr>_________________<br>
Devel mailing list<br>
<a href="mailto:Devel@ovirt.org">Devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/devel</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>Eyal Edri<br>Associate Manager</div><div>RHV DevOps<br>EMEA ENG Virtualization R&D<br>Red Hat Israel<br><br>phone: +972-9-7692018<br>irc: eedri (on #tlv #rhev-dev #rhev-integ)</div></div></div></div></div></div></div>
</div>