-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Black is not respecting max line length with string that use format function #3622
Labels
T: bug
Something isn't working
Comments
As you say, this is fixed in the preview style, and therefore likely to be in the stable style for next year. Our policy is not to change the style during the year, so there's nothing else to do here. |
Except it was already in preview last year :D Just hope you don't remove --experimental-string-processing this year and we are ok with it. But it is still a bug in 2023 style. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
We are using latest black 23.1.0 and it is putting .format after string making line longer than required.
Solution is to use experimental-string-processing however it will probably be removed due to its functionality moved to future.
However the problem is that future contains also other improvements and we don't want the style to change with each black upgrade.
To Reproduce
Black configuration:
Code runned with black (note line is 125 lines long and too long and flake8 checking breaks):
Code runned with black (experimental-string-processing = true):
And run it with these arguments:
Expected behavior
I should get line no longer than 120 chars
Environment
The text was updated successfully, but these errors were encountered: