This is a separate fix for pre-commit issues since the pre-commit CI will otherwise commit them to each pull request.
I could not fix every issue yet because some aren't as simple as they look. Also, I haven't been able to run pytest locally to check if I haven't broken anything.
Please take note of the flake8 configuration. I ignored the "B950 line too long" for migration files since they are automatically generated files. Let me know if you agree.
Notes
I originally started off with updated hooks (pre-commit run autoupdate), but that led to even more code changes. I think it's wise to run the hook update and subsequent code fixes separately.
MHLut
commented
6 months ago
This is a separate fix for pre-commit issues since the pre-commit CI will otherwise commit them to each pull request.
I could not fix every issue yet because some aren't as simple as they look. Also, I haven't been able to run pytest locally to check if I haven't broken anything.
Related to https://github.com/djangonaut-space/wagtail-indymeet/issues/339.
Notable changes
Please take note of the flake8 configuration. I ignored the "B950 line too long" for migration files since they are automatically generated files. Let me know if you agree.
Notes
I originally started off with updated hooks (
pre-commit run autoupdate), but that led to even more code changes. I think it's wise to run the hook update and subsequent code fixes separately.