search

techtonik / python-patch

Library to parse and apply unified diffs
https://pypi.python.org/pypi/patch
106 stars 63 forks source link

flake8 gradual linting #46

Open graingert opened 7 years ago

graingert commented 7 years ago

Currently the styleguide is not PEP8 complaint, flake8 however provides additional static analysis for unused variables etc.

All existing errors should be ignored so that new errors cannot be introduced.

Once this is done, each existing error can be fixed one at a time.

techtonik commented 7 years ago

Is there any way to just expose settings for static analysis?

As for styleguide, I like this point https://www.python.org/dev/peps/pep-0008/#a-foolish-consistency-is-the-hobgoblin-of-little-minds

graingert commented 7 years ago

yes, you disable the various errors that happen

techtonik commented 7 years ago

I mean expose those setting on the project/library level, so that code is not modified with comments.

graingert commented 7 years ago

@techtonik yes

graingert commented 7 years ago

And also you need to know, that I was expelled from Python community, because I didn't want to follow the lemmings way as I put it.

I consider myself a member of this community. Thanks for pointing this out. Maybe you should add this to your README.md?