On Tue, Jul 9, 2013 at 1:28 PM, Kieran Spear <kispear at gmail.com> 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." > > Cheers, > Kieran > > > _______________________________________________ > OpenStack-dev mailing list > OpenStack-dev at lists.openstack.org > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20130709/390855fa/attachment.html>