<div>Hi, stackers</div><div><br></div><div>Recently I submitted a patch of re-ordered imports by full module path, as stated in HACKING.rst</div><a href="https://review.openstack.org/#/c/8666/">https://review.openstack.org/#/c/8666/</a><div>
<br></div><div>The full module path of a import is the absolute path of the module/object from its package root.</div><div><br></div><div>Therefore, the order of the modules/objects</div><div><br></div><div>from quantum.plugins.cisco.db.models import BASE</div>
<div>from quantum.plugins.cisco.db import models</div><div><br></div><div>should be</div><div><br></div><div>from quantum.plugins.cisco.db import models (quantum.plugins.cisco.db.models)</div><div><div>from quantum.plugins.cisco.db.models import BASE (quantum.plugins.cisco.db.models.BASE)</div>
<div><br></div></div><div>The re-ordering was done using a tool I wrote.</div><div><a href="https://review.openstack.org/#/c/8592/">https://review.openstack.org/#/c/8592/</a></div><div><br></div><div>Once the tool gets in openstack-common, it will be easier to check the import order of your patches before submitting them for review.</div>
<div><br></div><div>Cheers,</div><div>LZY</div>