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

Is the concatenating lines with strings intended like this? #205

Closed
obestwalter opened this issue May 12, 2018 · 2 comments
Closed

Is the concatenating lines with strings intended like this? #205

obestwalter opened this issue May 12, 2018 · 2 comments

Comments

@obestwalter
Copy link

obestwalter commented May 12, 2018

Operating system: Arch Linux
Python version: 3.6
Black version: 18.4a4
Does also happen on master: dunno - I don't even know if it is a bug or a feature yet.

With default line length black turns

fmt = (
    "%(asctime)s %(filename)s.%(funcName)s:%(lineno)d %(levelname)s: "
    "%(message)s"
)

into

fmt = (
    "%(asctime)s %(filename)s.%(funcName)s:%(lineno)d %(levelname)s: " "%(message)s"
)

I would expect to either leave it alone or concatenate the strings like:

fmt = (
    "%(asctime)s %(filename)s.%(funcName)s:%(lineno)d %(levelname)s: %(message)s"
)

Is this behaviour intended?

@zsol
Copy link
Collaborator

zsol commented May 12, 2018

Yes, in the future black will be able to merge these strings, see #26 for some more discussion around this

@zsol zsol closed this as completed May 12, 2018
@obestwalter
Copy link
Author

Hi @zsol thanks, I missed that one, when looking for open issues.

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

2 participants