Open trickeydan opened 4 years ago
flake8-commas
is covered by black
, which i'd definitely prefer.
The rest sound fine, although a PR with a reviewable diff is probably the best way to know for sure whether it's worthwhile.
flake8-commas
is covered byblack
, which i'd definitely prefer.
No, it isn't. black
enables this sort of thing:
def foo(
a, b, c
):
which flake8-commas
does not as there is a missing trailing comma.
I'd like to propose adding some extra flake8 plugins:
flake8-debugger
- Checks forpdb
/ipdb
statementsflake8-docstrings
- Especially important for #129flake8-commas
- Enforces commas, helps reduce merge conflicts in lists.flake8-todo
- Checks forTODO
statements. (I use these a lot when developing a PR)