[openstack-dev] [nova] tox.ini ignores are TODOs?

Moore, Ryan ryan.moore at hp.com
Thu Jul 10 15:46:08 UTC 2014


Hi

tox.ini has the line # The rest of the ignores are TODOs
How serious are we about getting rid of these ignores?

Is it valid to -1 a patch that does not pass an ignored rule? E.g. E121 indentation is not a multiple of four

If a patch is submitted that does not pass a currently ignored rule, then this means that we are getting further away from enabling this rule. As not only do we have to fix the existing lines breaking the rules, but the newly submitted lines as well.

What do you think?

Ryan







-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20140710/ef023f8e/attachment.html>


More information about the OpenStack-dev mailing list