On 07/09/2013 10:55 AM, Joe Gordon wrote: > > On Tue, Jul 9, 2013 at 1:28 PM, Kieran Spear <[email protected] > <mailto:[email protected]>> wrote: > > Hi all, > > There's a review up to make Horizon pass H304 (no relative imports): > > https://review.openstack.org/#/c/35664/ > > Old-style relative imports are dangerous, but I can't think of a > good enough reason to forbid new-style imports, e.g.: > > from .views import IndexView > > instead of > > from openstack_dashboard.dashboards.project.images_and_snapshots.views \ > import IndexView > > particularly in Horizon where nesting is deep and you otherwise end > up with many imports that won't fit into 80 characters. I think if > we're going to make the change above the benefits would need to > outweigh the effect on readability. > > There's some discussion on the review already, but I'd like to hear > from a more general audience. Thoughts? > > > > http://google-styleguide.googlecode.com/svn/trunk/pyguide.html#Imports > > "Do not use relative names in imports. Even if the module is in the same > package, use the full package name. This helps prevent unintentionally > importing a package twice." >
>From pep8: Relative imports for intra-package imports are highly discouraged. Always use the absolute package path for all imports. Even now that PEP 328 is fully implemented in Python 2.5, its style of explicit relative imports is actively discouraged; absolute imports are more portable and usually more readable. Monty _______________________________________________ OpenStack-dev mailing list [email protected] http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
