-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add Vim and Kate setup guide for ruff server
#11615
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I pushed some minor nits, and apart from that one question this is good to go. Thank you for working on this!
crates/ruff_server/docs/setup/VIM.md
Outdated
endfunction | ||
``` | ||
|
||
If you'd like to use Ruff exclusively for linting, formatting, and import organization, you can disable those capabilities for Pyright. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this requires an example configuration? Do you have that? If not, we can remove this part.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It took that from the Neovim setup guide. However, as noted in my post above, I don't use pyright personally, so I am not able to translate this part.
I am using ruff server
in combination with pylsp. If this is helpful for someone, I could add the configuration necessary for this combination.
ruff server
ruff server
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Summary
In the roadmap for
ruff server
support for vim and kate is listed. Therefore I added setup guides for them based on the neovim guide. As I don't use pyright I wasn't able to translate the corresponding part from the neovim guide.Test Plan
Doesn't apply.