Skip to content

Commit

Permalink
upgrades the vscode-languageserver reference implementation under the…
Browse files Browse the repository at this point in the history
… hood. seems to work fine for IPC in vscode-graphql at least!

hopefully this solves #2230 once and for all!
  • Loading branch information
acao committed Jul 10, 2022
1 parent 4e598a8 commit 9b92ec7
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
9 changes: 9 additions & 0 deletions .changeset/chatty-insects-poke.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
---
'graphql-language-service-server': minor
'graphql-language-service-cli': minor
'vscode-graphql': minor
---

upgrades the `vscode-languageserver` and `vscode-jsonrpc` reference implementations for the lsp server. seems to work fine for IPC in `vscode-graphql` at least!

hopefully this solves #2230 once and for all!
3 changes: 1 addition & 2 deletions packages/graphql-language-service-server/src/startServer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ import {
WorkspaceSymbolRequest,
createConnection,
Connection,
DidSaveTextDocumentParams,
} from 'vscode-languageserver/node';

import { Logger } from './Logger';
Expand Down Expand Up @@ -304,7 +303,7 @@ async function addHandlers({
);
connection.onNotification(
DidSaveTextDocumentNotification.type,
async (params: DidSaveTextDocumentParams) => {
async params => {
const diagnostics = await messageProcessor.handleDidOpenOrSaveNotification(
params,
);
Expand Down

0 comments on commit 9b92ec7

Please sign in to comment.