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
Ignore the new W504 error from flake8 to get the tests passing #39
Conversation
I'm very confused, it doesn't appear to be ignoring either of the errors it says it's ignoring? |
Not sure why this isn't working. Could try flipping the order of the ignores to see if that helps, but honestly it's only yelling about a few places so it might just be worth it to fix those. |
Switched order doesn't seem to affect it. Maybe the E501s can be fixed, but unfortunately I don't think the W504s can. W504 complains about a binary operator before a linebreak, there's also a warning about binary operators after a linebreak, so you need to ignore at least one of them! |
This seems relevant: tholo/pytest-flake8#8 (comment) |
Co-Authored-By: alex <alex.gaynor@gmail.com>
Co-Authored-By: alex <alex.gaynor@gmail.com>
Nice catch! (Also the "suggest changes" UI is slick!) |
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.
Thanks for catching and fixing this!
No description provided.