<div dir="ltr">This is a reasonable proposal. I agree.</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><br></div><div dir="ltr">- jlk</div></div></div></div></div>
<br><div class="gmail_quote">On Tue, Oct 24, 2017 at 4:00 PM, James E. Blair <span dir="ltr"><<a href="mailto:corvus@inaugust.com" target="_blank">corvus@inaugust.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I discussed this briefly with some folks in IRC and received support,<br>
but I thought it wise to bring to the mailing list.<br>
<br>
I think we should add E741 to the list of pep8 errors that we ignore as<br>
a matter of course in infra projects.<br>
<br>
This is a recently added change which forbids the use of variables<br>
named either "l", "I", or "O".<br>
<br>
The same upgrade also brought E722, which rejects bare "except:"<br>
clauses.  There is a good reason to do so -- the KeyboardInterrupt<br>
exception does not inherit from the "Exception" class, and you almost<br>
always don't want to catch it.  So all such instances should be replaced<br>
with "except Exception:".  I think we should simply fix these errors.<br>
<br>
-Jim<br>
<br>
______________________________<wbr>_________________<br>
OpenStack-Infra mailing list<br>
<a href="mailto:OpenStack-Infra@lists.openstack.org">OpenStack-Infra@lists.<wbr>openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-infra</a></blockquote></div><br></div>