[issue29103] Make enum.py pep8 compliant

Raymond Hettinger report at bugs.python.org
Fri Dec 30 00:19:15 EST 2016


Raymond Hettinger added the comment:

I really don't like how this patch is done or even the spirit of it.  To my eyes, some of the code looks worse (especially the addition of trailing backslashes and some odd line-wraps that aren't sensitive to readability or context).

I'm going to close this.   If Ethan wants to make minor tweaks to his whitespace or line-wrapping, I believe he can do a better job with his own code.

Jean-Sebastien, thank you for trying to contribute, but PEP-8ing other people's code generally isn't helpful.  I recommend that you find a tracker item that needs a patch for a substantive change and work on that.  I think you will find it more satisfying and will have a much greater chance of acceptance.

----------
nosy: +rhettinger
resolution:  -> rejected
status: open -> closed

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue29103>
_______________________________________


More information about the Python-bugs-list mailing list