[openstack-dev] [Horizon] "import only module" message and #noqa
Gabriel pettier
gabriel.pettier at cloudwatt.com
Mon Dec 30 15:10:28 UTC 2013
So Tatiana pointed
http://lists.openstack.org/pipermail/openstack-dev/2013-June/thread.html#9993
to me, and from there i went on to read
http://lists.openstack.org/pipermail/openstack-dev/2013-August/thread.html#13074
and i can see valid points for H302, even if it annoy me sometime, if
it's better for reviews, i understand.
Sorry for the noise
On Mon, Dec 30, 2013 at 03:43:03PM +0100, Gabriel pettier wrote:
> Hi
>
> Reading horizon's code and recent reviews, i'm under the impression that
> it's a common practice to use #noqa to bypass the "import only modules"
> qa message, i'm unconvinced of the advantages of this policy (i think
> the namespace is often cleaner when one import only the symbols needed
> from the modules), so i think this policy could be removed, by adding
> "H302" to the list of ignored errors in tox.ini.
>
> This would allow removing a lot of #noqa comments, making for cleaner
> code.
>
> If there are significant advantages to this policy, however, it should
> be made more consistently applied to fix all these imports.
>
> Regards
>
> --
> Gabriel Pettier
> Software Engineer at CloudWatt.com
> 06 85 10 36 34
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
--
Gabriel Pettier
Software Engineer at CloudWatt.com
06 85 10 36 34
More information about the OpenStack-dev
mailing list