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

Update metals-languageclient #416

Open
ckipp01 opened this issue Aug 6, 2021 · 1 comment
Open

Update metals-languageclient #416

ckipp01 opened this issue Aug 6, 2021 · 1 comment
Labels
help wanted Extra attention is needed task General task that needs to be completed

Comments

@ckipp01
Copy link
Member

ckipp01 commented Aug 6, 2021

Task

When coc-metals and metals-languageclient were first written they were compliant with the LSP 3.15.0 spec. Now that we are on 3.16.0, coc.nvim has update here neoclide/coc.nvim@70a7903 and metals-languageclient here scalameta/metals-languageclient@5980c67. We should update our version of metals-languageclient to the latest and also make any changes necessary to work with 3.16.0.

Additional context

You can see what was needed in the VS Code extension to do this update here: scalameta/metals-vscode@7c46dca

Search terms

3.16.0
@ckipp01 ckipp01 added the task General task that needs to be completed label Aug 6, 2021
@ckipp01 ckipp01 added the help wanted Extra attention is needed label Mar 10, 2022
@ckipp01
Copy link
Member Author

ckipp01 commented Mar 10, 2022

Note that this is increasingly important now as the latest Metals is being built with 2.13 instead of 2.12. In order for coc-metals correctly pull the right version of Metals this update needs to happen or you won't be able to use the latest snapshots or the next stable version.

@ckipp01 ckipp01 pinned this issue Mar 10, 2022
@ckipp01 ckipp01 mentioned this issue Mar 15, 2022
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
help wanted Extra attention is needed task General task that needs to be completed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ckipp01