Lint project code #1580
Lint project code #1580
Conversation
|
Currently this is failing due to this:
|
|
Looks like it was in the last commit -- On Fri, Aug 21, 2015 at 4:25 PM, Anthony [email protected] wrote:
Eric Holscher |
This lints project code to medium strictness and conforms to pep257. This uses some additional options to reduce false errors, which I'll add in a separate PR. This should pass our normal linting without a problem, except the minor change to unused variable name regex. This adds `__` to the default list of `_` and `dummy*` for unused arguments. I prefer the default of `_`, but that overlaps with the translation helper `_()`. I'd be happy to rethink a scheme here.
agjohnson
force-pushed
the
lint-project
branch
from
deaa316 to
3fcf523
Compare
This lints project code to medium strictness and conforms to pep257. This uses
some additional options to reduce false errors, which I'll add in a separate PR.
This should pass our normal linting without a problem, except the minor change
to unused variable name regex.
This adds
__to the default list of_anddummy*for unused arguments. Iprefer the default of
_, but that overlaps with the translation helper_().I'd be happy to rethink a scheme here.
Fixes #1472, refs #1474