-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Recommended approach with RLS #1710
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
Comments
I think you should use the cargo watch integration |
Yeah, you should turn the RLS off and enable cargo-watch. To enable cargo watch, either answer "Yes" to the notification on project loading, or set the Closing, but feel free to reopen if this doesn't work! |
Does the |
Thanks for the fast reply. I did have cargo-watch enabled. I'm appreciative of rust-analyzer and the latency is awesome. For a relative newbie, a complete list of suggestions are v helpful. Is rust-analyzer expected to find as many suggestions as RLS? Or is the speed a tradeoff for completeness? And next time I find a case where rust-analyzer doesn't have suggestions, and RLS does - should I post an issue? |
yeah, ra with cargo watch should be equivalent to rls. Could it be the case
that RLS runs clippy by default as well, and we don't? It's possible to
override `cargo watch` command such that it triggers clippy.
…On Wed, 21 Aug 2019 at 19:15, Maximilian Roos ***@***.***> wrote:
Thanks for the fast reply. I did have cargo-watch enabled.
I'm appreciative of rust-analyzer and the latency is awesome. For a
relative newbie, a complete list of suggestions are v helpful. Is
rust-analyzer expected to find as many suggestions as RLS? Or is the speed
a tradeoff for completeness?
And next time I find a case where rust-analyzer doesn't have suggestions,
and RLS does - should I post an issue?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#1710>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AANB3M6RNPOLJVV3V6ZFV5TQFVSZBANCNFSM4IOBGJQA>
.
|
I do actually get clippy lint warnings. |
I would be surprised if there are many cases left where RLS (i.e. racer) gives better completions than RA. Feel free to open issues if you find such examples. |
Great, thanks @flodiebold |
Something off the top of my head:
|
This is another, I think? #1622 Would be great to have this from RLS and everything else from rust-analyzer, for example. But maybe that's best implemented as a setting in RLS to exclude everything else |
I switched to cargo-watch+clippy but I strongly miss instant diagnostics from RLS. Is there anything being worked on in that area? On Neovim I was using rust-analyzer through coc.nvim with diagnostics disabled, and RLS through ALE for diagnostics. FWIW I didn't enable diagnostics on every input, just when leaving insert mode and normal mode edits, which is a good balance, and better than being forced to save the file to sync diagnostics. |
I've looked for guidance elsewhere but haven't found anything, so thought it OK to venture an issue here.
What's the recommended approach in VSCode for working with RLS? Currently:
cargo check --all
Thank you for such an awesome tool - the latency makes rust so much nicer to work with!
The text was updated successfully, but these errors were encountered: