Skip to content
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 doesn't split the line with string and format() #581

Closed
jaklan opened this issue Oct 22, 2018 · 2 comments
Closed

Black doesn't split the line with string and format() #581

jaklan opened this issue Oct 22, 2018 · 2 comments

Comments

@jaklan
Copy link

jaklan commented Oct 22, 2018

Operating system: Ubuntu 16.04
Python version: 3.6.6
Black version: 18.9b0
Does also happen on master: yes

Length limit: 80

Before Black:

logging.info("Evaluating the model with {} of the supplied data held out for tests ({}).".format(sup_opts["test_split"], len(test)))

After Black:

logging.info(
    "Evaluating the model with {} of the supplied data held out for tests ({}).".format(
        sup_opts["test_split"], len(test)
    )
)

The line with string should be splitted, because it exceeds the line length limit, but it's not. The same effect is for different settings, e.g. 60 chars, 50 chars.

@JelleZijlstra
Copy link
Collaborator

This seems the same as #182.

@wbolster
Copy link
Contributor

this should be closed as a dup of #182 indeed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants