-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
preview option support #37
Comments
Sorry for the late reply, I was on vacation. Sadly, the problem here is that there is no way to pass this option to the blackd itself, see protocol documentation. :( I am thinking about making it possible via a commit there, but it may take some time. Until then the issue will be closed as not being the fault of the plugin itself. |
Ouch. I didn't know blackd didn't have support for this. It seems to be just an oversight on their part. |
Please do. As soon as it is merged I'll get right down to implementing it. |
Will be added in the next release of black. I think this can be reopened |
Black 22.8.0 was released with this addition, this feature can be implemented now 🚀🚀🚀 |
Hopefully this will be fixed soon. In fact, this is a blocking problem for me. |
|
Yes, hopefully I can get around to implementing that feature this week. |
I've just implemented it |
Requesting feedback for the PR: #52 |
Solved by #52 |
Could we have a new release with this fix? |
There is a possibility that a new patch-level release will be ready in a couple of days. Please be patient. |
@lensvol Sorry for pinging again. Could we have a release? |
sigh I really should do that, yes. Will try to implement support for 3.11 and release before this week ends. |
@Mogost It has been released. |
@lensvol thank you for your present for my birthday 😃 |
intellij-blackconnect does not respect blacks
preview
optionhttps://black.readthedocs.io/en/stable/the_black_code_style/future_style.html#preview-style
The text was updated successfully, but these errors were encountered: