-
Notifications
You must be signed in to change notification settings - Fork 52
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
Update to reflect the stabilized ruff server
#531
Conversation
8a696d8
to
ac08069
Compare
ruff server
ruff server
ac08069
to
267f362
Compare
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.
LGTM. Let's wait for @charliermarsh I think he wants to do the final approval.
Should we add a sentence saying that whether ruff-lsp or ruff server
is used, can be seen in the toolbar?
Good idea. I'll add a "Troubleshooting" section because I don't think it's present in the toolbar by default. Edit: Added a troubleshooting section in #533 |
267f362
to
419204b
Compare
Summary
This PR updates the README to reflect the stabilized
ruff server
. It also changes the minimum required version for the usage ofruff server
innativeServer: auto
to be0.5.3
.For the README, the settings that are just a pass through to the native server are removed and instead we point the users to the online settings reference. Now, the settings table only contains the ones which are specific to VS Code extension.
For review, here's the rendered README: https://github.com/astral-sh/ruff-vscode/blob/dhruv/readme/README.md
resolves: #520