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
DM-11658: Enable flake8 testing #14
Conversation
Remove the unused install_aliases() call as none of the packages were being imported that needed this alias.
setup.cfg
Outdated
@@ -0,0 +1,8 @@ | |||
[flake8] | |||
max-line-length = 110 | |||
ignore = E133, E226, E228, N802, N803, W503 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are these and why are they ignored?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They are the standard ignore codes documented in the developer guide.
https://developer.lsst.io/coding/python_style_guide.html#exceptions-to-pep-8
(although two of the ones in the dev guide aren't relevant here: I probably should add them in for consistency).
setup.cfg
Outdated
|
||
[tool:pytest] | ||
addopts = --flake8 | ||
flake8-ignore = E133 E226 E228 N802 N803 W503 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the repetition required?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. Unfortunately.
No description provided.