-
Notifications
You must be signed in to change notification settings - Fork 894
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
remove rls support #3316
Comments
The In its default configuration in emacs 29, eglot knows that both rls and rust-analyzer exist, sees that both are available on PATH, and prompts the user rather than just choosing to run rust-analyzer. |
I'd blame that on eglot, it shouldn't be trying to run rls. Consider filing an issue there. (That's not to say the RA proxy itself isn't annoying if you compile your own RA, but it's the best way we have to make rustup-managed RA work. But that's a completely different topic.) |
Eglot upstream removed the reference to |
I believe all work required in this repo has concluded in #3920. As such, I'm closing this issue as completed now. For further work, please refer to rust-lang/rust#126856. |
With rust-analyzer taking over, should we remove the rls specific tests, and/or the rls proxy?
The text was updated successfully, but these errors were encountered: