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

Elm Extension won't start because it requires VSCode 1.42 #7243

Closed
JesterOrNot opened this issue Feb 28, 2020 · 9 comments · Fixed by #7564
Closed

Elm Extension won't start because it requires VSCode 1.42 #7243

JesterOrNot opened this issue Feb 28, 2020 · 9 comments · Fixed by #7564
Labels
help wanted issues meant to be picked up, require help vscode issues related to VSCode compatibility

Comments

@JesterOrNot
Copy link

Description

Language server fails to start due to an outdated version of VSCode

Reproduction Steps

image
Open https://gitpod.io/#https://github.com/jesterornot/gitpod-elm/tree/bug
The relevant extension is https://marketplace.visualstudio.com/items?itemName=Elmtooling.elm-ls-vscode

OS and Theia version:
Gitpod

Diagnostics:

The language client requires VS Code version ^1.42.0 but received version 1.41.1
@vince-fugnitto
Copy link
Member

@JesterOrNot you can specify a VSCODE_API_VERSION when starting the application if you'd like, else I'd suggest using an older version of the elm extension. We are still working towards supporting newer versions of the API (and ultimately extensions). (related #6804)

For the moment I think we can hold-off creating issues for each extension we encounter which requires the newer API as it will become noisy and instead work towards supporting the new API.
In the meantime it is acceptable to use older versions of extensions (it is actually what we do with the builtins).

@vince-fugnitto vince-fugnitto added the vscode issues related to VSCode compatibility label Feb 28, 2020
@akosyakov
Copy link
Member

akosyakov commented Feb 29, 2020

We should bump up VSCODE_API_VERSION. It's again vscode-languageclient defaults to 1.42. Any extension which is using this version regardless whether it actually uses unsupported by us features won't work.

After that we should test elm for unsupported API.

We are still working towards supporting newer versions of the API (and ultimately extensions). (related #6804)

that's not necessary unrelated, monaco-languageclient upgrade is needed only for language support via Theia extensions

@akosyakov akosyakov added the help wanted issues meant to be picked up, require help label Feb 29, 2020
@akosyakov
Copy link
Member

Yes, just checked that they are using progress enhancements from new vscode-languageclient: elm-tooling/elm-language-client-vscode@b6516fb#diff-bc3c0f2fe3aca9ddbe96347ffb00641e

It is already supported by us.

@akosyakov
Copy link
Member

@JesterOrNot it can be bumped up here:

export const VSCODE_DEFAULT_API_VERSION = '1.41.1';

@JesterOrNot
Copy link
Author

On it!

JesterOrNot pushed a commit to JesterOrNot/theia that referenced this issue Feb 29, 2020
@dsseng
Copy link
Contributor

dsseng commented Apr 10, 2020

Vetur is also affected, v0.24.0 doesn't work without VS Code API v1.42.x

@akosyakov
Copy link
Member

@sh7dm Could you send a PR bumping up the version and check whether vetur works with it?

@dsseng
Copy link
Contributor

dsseng commented Apr 14, 2020

Of course 🤠 !

@dsseng
Copy link
Contributor

dsseng commented Apr 14, 2020

@akosyakov that works, I'll submit a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted issues meant to be picked up, require help vscode issues related to VSCode compatibility
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants