We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Check this note from the flake8 docs:
Following the recommended settings for Python’s configparser, Flake8 does not support inline comments for any of the keys. So while this is fine: [flake8] per-file-ignores = # imported but unused init.py: F401 this is not: [flake8] per-file-ignores = init.py: F401 # imported but unused
Following the recommended settings for Python’s configparser, Flake8 does not support inline comments for any of the keys. So while this is fine:
[flake8] per-file-ignores = # imported but unused init.py: F401
this is not:
[flake8] per-file-ignores = init.py: F401 # imported but unused
However, that's exactly what we do:
pandas/setup.cfg
Lines 71 to 104 in 98e2229
In each case, the comment should be moved to be on its own line, on the line before the code
Else, we risk accidentally putting something in the comments that'll mess with how flake8 works
The text was updated successfully, but these errors were encountered:
I would like to take this one.
Sorry, something went wrong.
awesome, PRs welcome
RhnSharma
Successfully merging a pull request may close this issue.
Check this note from the flake8 docs:
However, that's exactly what we do:
pandas/setup.cfg
Lines 71 to 104 in 98e2229
In each case, the comment should be moved to be on its own line, on the line before the code
Else, we risk accidentally putting something in the comments that'll mess with how flake8 works
The text was updated successfully, but these errors were encountered: