Skip to content

docs: Improve the rustaceanvim documentation#168

Merged
mrjones2014 merged 1 commit into
mrjones2014:masterfrom
cormacrelf:rust-config
May 8, 2026
Merged

docs: Improve the rustaceanvim documentation#168
mrjones2014 merged 1 commit into
mrjones2014:masterfrom
cormacrelf:rust-config

Conversation

@cormacrelf
Copy link
Copy Markdown
Contributor

@cormacrelf cormacrelf commented May 5, 2026

  1. "out of the box" misled me when I skipped to the rustaceanvim section, thought it meant "you don't have to do anything". You do have to add a before_init.
  2. Some settings need to be provided in the initparams for rust-analyzer. They are used by people who use non-cargo build systems (hello!). I have updated one of the examples but I think it should maybe be in the main example and all other ones? Other LSPs might need this too.
  3. the vim.lsp.config("rust_analyzer"...) was wrong, it's rust-analyzer.

@cormacrelf cormacrelf changed the title Improve the rustaceanvim documentation docs: Improve the rustaceanvim documentation May 5, 2026
@cormacrelf cormacrelf force-pushed the rust-config branch 3 times, most recently from 517f704 to 61f08ae Compare May 5, 2026 07:58
Comment thread README.md Outdated
@mrjones2014
Copy link
Copy Markdown
Owner

Thanks for contributing!

@cormacrelf
Copy link
Copy Markdown
Contributor Author

I've decided I don't think we can assume all LSP servers will tolerate initializationOptions being set to the same value as the configuration request. In the case of RA it is supported and required for some config options to work, but LSP protocol makes absolutely no comment on the contents of initializationOptions so that's not generalizable.

Will clean up tomorrow

@cormacrelf
Copy link
Copy Markdown
Contributor Author

+ just applied, and reworded the recommended before_init introduction.

@mrjones2014 mrjones2014 merged commit 29e28f0 into mrjones2014:master May 8, 2026
4 checks passed
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.

2 participants