Skip to content

Update vscode-languageclient to 7.0.0 - #432

Merged
angelozerr merged 1 commit into
redhat-developer:masterfrom
datho7561:linked-editing-support
Apr 16, 2021
Merged

angelozerr merged 1 commit into
redhat-developer:masterfrom
datho7561:linked-editing-support

Conversation

@datho7561

Copy link
Copy Markdown
Contributor

This allows for textDocument/linkedEditingRange to be used.

Replaces #426

Signed-off-by: David Thompson [email protected]

@angelozerr

Copy link
Copy Markdown
Contributor

I think you can switch your draft PR in real PR.

@datho7561
datho7561 marked this pull request as ready for review April 14, 2021 13:13
@datho7561
datho7561 force-pushed the linked-editing-support branch from 533ff94 to f8ba051 Compare April 14, 2021 13:20
This allows for `textDocument/linkedEditingRange` to be used.

Replaces redhat-developer#426

Signed-off-by: David Thompson <[email protected]>
@datho7561
datho7561 force-pushed the linked-editing-support branch from f8ba051 to 0c3516f Compare April 14, 2021 19:44
@angelozerr

Copy link
Copy Markdown
Contributor

I'm really annoying because this PR works good with master of LemMinx but with the Leinked Editing range PR eclipse-lemminx/lemminx#991 (even if linked editing range is disable) the server crashes (no completion, no validation, etc)

@datho7561

Copy link
Copy Markdown
Contributor Author

I also get this. I'll play around with this a bit today to see if I can figure out what's going on.

@angelozerr
angelozerr merged commit c248ffb into redhat-developer:master Apr 16, 2021
@angelozerr

Copy link
Copy Markdown
Contributor

This PR works great with LemMinx and with LSP4J 0.9 (master) and 0.11 but not for 0.12. As it's not a problem coming from this PR, I merge it. If 0.12 have a problem we can use 0.11 for linked editing range support.

Thanks @datho7561 !

@angelozerr angelozerr added this to the 0.17.0 milestone Apr 16, 2021
@datho7561
datho7561 deleted the linked-editing-support branch April 16, 2021 13:48
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