Skip to content

Update $/cancelRequest to send object#3

Open
alissa-tung wants to merge 1 commit intocodemirror:mainfrom
alissa-tung:patch-1
Open

Update $/cancelRequest to send object#3
alissa-tung wants to merge 1 commit intocodemirror:mainfrom
alissa-tung:patch-1

Conversation

@alissa-tung
Copy link

I am not sure the code formatting that you prefer, I learned from the following existed code:

this.notification<lsp.DidCloseTextDocumentParams>("textDocument/didClose", {textDocument: {uri}})

This fix addresses the LSP:

interface CancelParams {
	/**
	 * The request id to cancel.
	 */
	id: integer | string;
}

See https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#cancelRequest

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