Hi, I discussed this briefly with some folks in IRC and received support, but I thought it wise to bring to the mailing list.
I think we should add E741 to the list of pep8 errors that we ignore as a matter of course in infra projects. This is a recently added change which forbids the use of variables named either "l", "I", or "O". The same upgrade also brought E722, which rejects bare "except:" clauses. There is a good reason to do so -- the KeyboardInterrupt exception does not inherit from the "Exception" class, and you almost always don't want to catch it. So all such instances should be replaced with "except Exception:". I think we should simply fix these errors. -Jim _______________________________________________ OpenStack-Infra mailing list [email protected] http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra
