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

Change WorkspaceEdit to use documentChanges #23

Merged
merged 10 commits into from
Mar 11, 2024
Merged

Conversation

lieryan
Copy link
Member

@lieryan lieryan commented Mar 10, 2024

There are two variants of WorkspaceEdit, this change generates the WorkspaceEdit.documentChanges and converts it to changes for clients that doesn't support documentChanges.

Whether a client supports versioned document edits is expressed via wworkspace.workspaceEdit.documentChanges` client capability.

@lieryan lieryan self-assigned this Mar 10, 2024
@lieryan lieryan force-pushed the lieryan-document-changes branch from 9dee032 to 40c9c8d Compare March 11, 2024 12:36
lieryan added 5 commits March 11, 2024 23:45
…ument

This argument selects between returning WorkspaceEditWithChanges or
WorkspaceEditWithDocumentChanges variants of WorkspaceEdit.
We need to upgrade this at some point.
@lieryan lieryan force-pushed the lieryan-document-changes branch from 5e259a6 to 0d32835 Compare March 11, 2024 14:13
@lieryan lieryan force-pushed the lieryan-document-changes branch from 2935d13 to adab2ca Compare March 11, 2024 14:17
@lieryan lieryan merged commit 0bcf4d3 into main Mar 11, 2024
5 checks passed
@lieryan lieryan deleted the lieryan-document-changes branch March 11, 2024 14:18
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.

1 participant