Open Stack

On 2017-10-24 16:00:25 -0700 (-0700), James E. Blair wrote:
[...]
> 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".

Yes, straight from the I-know-what-you-were-probably-thinking-but-
you're-doing-it-wrong department. I have no interest in protecting
incompetent developers from their poor choice of fonts. This has
been a solved problem for decades, perhaps centuries.

> 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.

Sounds good, I concur.
-- 
Jeremy Stanley
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 949 bytes
Desc: Digital signature
URL: <http://lists.openstack.org/pipermail/openstack-infra/attachments/20171024/0c94d812/attachment-0001.sig>