On Tue, 2014-12-09 at 13:46 -0500, Sean Dague wrote: > Yes, the following fails H305 and H306. > > nova/tests/fixtures.py > > """Fixtures for Nova tests.""" > from __future__ import absolute_import > > import gettext > import logging > import os > import uuid > > import fixtures > from oslo.config import cfg > > from nova.db import migration > from nova.db.sqlalchemy import api as session > from nova import service > > > Because name normalization is hard (fixtures is normalized to > nova.tests.fixtures so H305 thinks it should be in group 3, and H306 > thinks it should be after oslo.config import cfg). > > To sort things you have to normalize them.
I agree you have to normalize imports to sort them, but to my mind the appropriate normalization here is purely textual; we shouldn't be expecting any relative imports (and should raise an error if there are any). Still, that does show that some work needs to be done to the simpler H306 test (probably involving changes to the core import normalization)… -- Kevin L. Mitchell <kevin.mitch...@rackspace.com> Rackspace _______________________________________________ OpenStack-dev mailing list OpenStackemail@example.com http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev