-
Notifications
You must be signed in to change notification settings - Fork 77
Add Rust Analyzer in the install instructions #27
Add Rust Analyzer in the install instructions #27
Conversation
Is this a common problem? My understanding is that effectively all Rust developers on VS Code use |
Either this change or the install instructions should contain rust analyzer. |
Recommending installing As an aside, do you have a link to where someone was having trouble? I don't think I saw it, but I'll offer help if I'm able, and seeing what's hard can drive development. |
I'll make that PR later.
https://discord.com/channels/1054443721975922748/1089255512228499516/1089415250740260915
…On Sun, 26 Mar 2023, 08.37 Nathan Varner, ***@***.***> wrote:
Recommending installing rust-analyzer would be better, since Rust
development without it would be difficult.
As an aside, do you have a link to where someone was having trouble? I
don't think I saw it, but I'll offer help if I'm able, and seeing what's
hard can drive development.
—
Reply to this email directly, view it on GitHub
<#27 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAIDVSFW4SWG64RALCDX4HLW57P3LANCNFSM6AAAAAAWHWJOXE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
What's the benefit of having the cargo build task depend on rust analyzer rather than this? |
Oh, I see; it's not possible to even run the extension without the RA plugin. However, using it gives us problem matchers. In general, it should usually provide better editor integration than a command. It probably also works better in strange environments. For instance, I can see someone alias an unusual command to I think it's still best to pivot this PR into recommending installing RA. It would also be okay if there's an easy way to add the RA problem matcher to the command line task without depending on RA, and the task is renamed to indicate that it's part of the build process for testing the extension, not a recommended way of building the LSP. |
And thanks for providing support! |
Thanks! And no problem.. Unfortunately collaboration requires that someone gets educated sometimes. |
5a57810
to
893b324
Compare
@nvarner It should be ready now. |
You can also add |
for this project.
Perfect suggestion; I've added it now. |
Thanks! This will help new users. |
UPDATE: This PR now specifies the dependence on Rust Analyzer through the install instructions.
Currently, installing the extension for developers, depends on Rust Analyzer.
That is due to
"type": "cargo"
is a construction from rust-analyzer.Instead, this just invokes cargo as a shell command.