Skip to content
This repository was archived by the owner on Nov 18, 2022. It is now read-only.

detect missing RLS component and give a useful message #208

Open
nrc opened this issue Dec 3, 2017 · 3 comments
Open

detect missing RLS component and give a useful message #208

nrc opened this issue Dec 3, 2017 · 3 comments
Labels
enhancement Indicates new feature requests
Milestone

Comments

@nrc
Copy link
Member

nrc commented Dec 3, 2017

If the rls-preview component is missing (which happens fairly often), we should advise the user that this is the case, explain why it happens, and suggest a remedy (downgrade or use beta/stable).

@nrc nrc added this to the impl period milestone Dec 3, 2017
@ejmahler
Copy link

ejmahler commented Dec 3, 2017

This should supersede #203 imo

@mattico
Copy link

mattico commented Jan 23, 2018

Even better, suggest and install the previous nightly which works: https://www.reddit.com/r/rust/comments/7sds1a/atom_iderust_011_handling_nightly_does_not/

@Xanewok
Copy link
Member

Xanewok commented Apr 7, 2019

FWIW during the component installation we now present the rustup component add rls in the task view, along with the output. This means that the user sees what's wrong, however it might be good to follow Atom in its footsteps and suggest using the previous nightly which includes the component.

@Xanewok Xanewok added the enhancement Indicates new feature requests label Apr 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Indicates new feature requests
Projects
None yet
Development

No branches or pull requests

4 participants