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

Docs: Using rust-analyzer from rustup #493

Merged
merged 1 commit into from
Feb 15, 2023

Conversation

psibi
Copy link
Collaborator

@psibi psibi commented Feb 14, 2023

Also discourage users from using rls

Also discourage users from using rls
@psibi psibi requested a review from brotzeit February 14, 2023 04:37
Copy link
Owner

@brotzeit brotzeit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

related code was moved to the lsp-mode repo. `rustic-lsp-server` sets
the value of `lsp-rust-server`.
rust-analyzer is the default and can be changed to rls if required (Note that rls
is deprecated and is slated to be removed). lsp-mode related code was

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/slated/stated/ ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do mean slated to indicate that any feature related to rls is expected to be removed in future.

@psibi psibi merged commit 39423d1 into brotzeit:master Feb 15, 2023
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

Successfully merging this pull request may close these issues.

3 participants