-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
flake8 error E231 after a successful black run #1289
Comments
This is a bug related to #1288 and will be fixed as part of that issue. Thank you for the time to report this! In the mean time, if you remove the trailing comma by hand, Black won't re-add it. |
This is still reproducible despite the #1288 has been closed. Would it be possible to re-open this issue? |
Hi! I tried it on the playground, but it's formatted correctly. Do you have a failing example? (meaning unmodified but would expect modifications) |
@felix-hilden My bad. I used black 19.10b0 and indeed in the most recent 22.1.0 this problem is resolved. |
To Reproduce Steps to reproduce the behavior:
example.py
flake8
on it:Expected behavior A clear and concise description of what you expected to happen.
Expecting not to get any
flake8
errors barring the different line length setting, and edge cases where black is unable to condense output under 88 chars.Environment (please complete the following information):
Also getting the same behaviour with
black-19.10b0
.The text was updated successfully, but these errors were encountered: