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

Add lsp command to fix rust-analyzer #1026

Merged
merged 1 commit into from
Jun 17, 2022

Conversation

jackos
Copy link
Contributor

@jackos jackos commented Jun 16, 2022

This is continuing from #911, as requested I've moved functionality into a subcommand that can be run like rustings lsp and added documentation plus added it to the initial welcome message.

This will ensure a good developer experience for newcomers to the Rust ecosystem.

Copy link
Member

@shadows-withal shadows-withal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! Just two little nits.

src/main.rs Outdated Show resolved Hide resolved
src/main.rs Outdated Show resolved Hide resolved
@jackos
Copy link
Contributor Author

jackos commented Jun 16, 2022

Thank you! Just two little nits.

Thanks for that, I squashed in the changes

@shadows-withal shadows-withal merged commit 294ef8d into rust-lang:main Jun 17, 2022
@shadows-withal
Copy link
Member

@all-contributors please add @jackos for code

@allcontributors
Copy link
Contributor

@diannasoreil

I've put up a pull request to add @jackos! 🎉

dmoore04 pushed a commit to dmoore04/rustlings that referenced this pull request Sep 11, 2022
MadLittleMods added a commit to MadLittleMods/rustlings that referenced this pull request Oct 14, 2024
…e repo directly

> Yes, you are right, if you just clone the repository and try to edit the exercises, the language server will not work. This is one downside of the current approach. But this only affects developing exercises.
>
> The new method of doing Rustlings is to install Rustlings using `cargo install rustlings` (not published yet), then running `rustlings init`. No repo cloning happens. Instead, the directory `rustlings/` will be created where you find the exercises. The language server works there out of the box :)
>
> I need to add a warning when people try to work on the exercises from the repository. Thanks pointing this out.
>
> -- @mo8it, rust-lang#1935 (comment)

Other references:

 - Previous `rustlings lsp` command: rust-lang#1026
 - The changelog says "LSP support out of the box", https://github.com/rust-lang/rustlings/blob/main/CHANGELOG.md#lsp-support-out-of-the-box
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