-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Update rls-data and rls-analysis versions #53440
Comments
Also /cc @emilio since he also made changes to rls-analysis and might be interested in this. |
This part 2/3 for fixing rust-lang/rust#53440.
This part 2/3 for fixing rust-lang/rust#53440.
This part 2/3 for fixing rust-lang/rust#53440.
I guess updating it in the compiler before rls was the way to go? I submitted rust-lang/rls#973 but never got an answer about how to do this. Thanks for doing this! |
I've never done it before, but that's what the documentation seems to imply. AFAICT step 1 is land it in rustc, step is land the RLS change, and step 3 is update the submodule in rustc to point to the new RLS. |
oh crap, sorry!
That's correct. |
This part 1/3 for fixing rust-lang#53440.
Update version of rls-data used with save-analysis This part 1/3 for fixing rust-lang#53440.
Parts 1 and 2 have landed, so I made a PR for part 3. #53649 |
This is done. |
I'd like to bump rls-analysis and rls-data to the latest available versions (0.16 and 0.18, respectively). Specifically to pick up the changes from rust-dev-tools/rls-analysis#147. It looks like this will need the three-step update process described at /~https://github.com/rust-lang/rust/blob/master/CONTRIBUTING.md#breaking-tools-built-with-the-compiler. I will attempt to create the appropriate patches.
/cc @nrc
The text was updated successfully, but these errors were encountered: