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

Change rls recommendation to rust-analyzer #321

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 4 additions & 15 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -148,20 +148,9 @@ There are similar commands for building the Theseus book:
Our personal preference is to use Visual Studio Code (VS Code), which has excellent, official support from the Rust language team. Other options are available [here](https://areweideyet.com/), but we don't recommend them.

For VS Code, recommended plugins are:
* Rust (rls), by rust-lang
* Better TOML, by bungcip
* x86 and x86_64 Assembly, by 13xforever

### Fixing RLS Problems
Sometimes RLS just doesn't want to behave, especially if the latest Rust nightly version had build errors. In that case, try the following steps to fix it:
* Set your default Rust toolchain to the one version in the `rust-toolchain` file, for example:
`rustup default nightly-2019-07-09`.
* With your newly-set default toolchain, add the necessary components:
`rustup component add rls rust-analysis rust-src`.
* In VS Code (or whatever IDE you're using), uninstall and reinstall the RLS extension, reloading the IDE each time.
* Check your IDE's settings to make sure that no weird rust or RLS settings have been set; building Theseus doesn't require any special RLS settings.
* If you're still having lots of issues, remove all other toolchains besides the default one and try again.
You can see other installed toolchains with `rustup toolchain list`.
* [rust-analyzer](https://marketplace.visualstudio.com/items?itemName=matklad.rust-analyzer), by matklad
* [Better TOML](https://marketplace.visualstudio.com/items?itemName=bungcip.better-toml), by bungcip
* [x86 and x86_64 Assembly](https://marketplace.visualstudio.com/items?itemName=13xforever.language-x86-64-assembly), by 13xforever


# Other
Expand All @@ -188,4 +177,4 @@ We adhere to similar development and style guidelines as the main Rust project.

PRs and issues are welcome from anyone; because Theseus is an experimental OS, certain features may be deprioritized or excluded from the main branch. Don't hesitate to ask or mention something though! :smile:

More details to come.
More details to come.