-
Notifications
You must be signed in to change notification settings - Fork 36
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
Support remote blackd server #160
Comments
The extension runs Black locally via LSP for performance reasons. This would be a feature request to use |
Hi @karthiknadig, is there anything in particular you're looking for from me, or are you waiting to see if others share this need? |
We usually leave feature requests open for some time to see if there is any community interest. Since this is a large change and we want to make sure there is demand for this. |
Thanks for the feature request! We are going to give the community 60 days from when this issue was created to provide 5 👍 upvotes on the opening comment to gauge general interest in this idea. If there's enough upvotes then we will consider this feature request in our future planning. If there's unfortunately not enough upvotes then we will close this issue. |
Our team is using a remote blackd server, that's kept in sync with the version in CI. Some of our team members use VS Code, others use PyCharm. The latter has a plugin that let's the users set the path to a remote version of black, so that not everyone has to have a running version of the black server running and needs to keep track of that when we do a version bump on CI. It'd be great if this extension could add support for remote servers, since it's the biggest one for formatting with black.
I'm not sure how this extension currently handles black interaction (with blackd, it would look something like
curl -s -XPOST "<HOST>:<PORT>" -d "<YOUR_PYTHON_CODE>"
) and configuration (like line-length). Is the configuration passed with every request as with blackd or covered somehow differently? If that could be reused, that would be of course very nice.The text was updated successfully, but these errors were encountered: