You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
So I've started removing the Flake8 pre-commit hook...it is just too militant, and the fact I can't commit any line over 79 characters really isn't improving my code an awful lot, especially as we already have Black.
I was wondering if we could consider which pre-commits were necessary, and remove the rest? Commit failing is a pain.
The text was updated successfully, but these errors were encountered:
I believe the flake8 settings are already set up for compatibility with black - see the .flake8 file in the {{ cookiecutter.repo_name }} folder of the main branch.
@AndreasThinks - not necessarily a solution to your problem, but you should be able to modify the configuration of black to change the line width.
Regarding flake8 itself, it runs additional PEP8 checks not covered by black, which only really covers formatting. For example, I don't think black picks up on unused imports, but flake8 will!
So I've started removing the Flake8 pre-commit hook...it is just too militant, and the fact I can't commit any line over 79 characters really isn't improving my code an awful lot, especially as we already have Black.
I was wondering if we could consider which pre-commits were necessary, and remove the rest? Commit failing is a pain.
The text was updated successfully, but these errors were encountered: