Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Numerous flake8 lint errors are temporarily disabled #366

Open
alexdunnjpl opened this issue Sep 16, 2022 · 1 comment
Open

Numerous flake8 lint errors are temporarily disabled #366

alexdunnjpl opened this issue Sep 16, 2022 · 1 comment
Assignees

Comments

@alexdunnjpl
Copy link
Contributor

alexdunnjpl commented Sep 16, 2022

🐛 Describe the bug

The following flake8 lint errors have been disabled to allow branch integration testing to pass without being flooded.
B007, B008, B902, B950, D100, D101, D102, D103, D104, D107, D200, D202, D205, D212, D402, D403, D415, D417, E741, F401, F841, N805, N818

The codebase must be linted and brought into compliance, and these errors re-enabled in setup.cfg's flake8 section.

🕵️ Expected behavior

With the temporary extend-ignore removed and the permanent extend-ignore reinstated, tox's lint testing passes.

@alexdunnjpl
Copy link
Contributor Author

alexdunnjpl commented Sep 19, 2022

@jordanpadams running both black and autopep8 result in minimal changes (not sure why since I assume flake8 is assessing PEP8 conformity).

Manually fixing the >1100 errors manually is obviously not a smart play right now.

I'll loop back to this when I have some free time, but will assume it's not a high priority unless you tell me otherwise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ToDo
Development

No branches or pull requests

2 participants