Skip to content

Aligns language server with protocol proposition#611

Merged
dbaeumer merged 1 commit intomicrosoft:masterfrom
giltho:specified-logtrace
Jun 26, 2020
Merged

Aligns language server with protocol proposition#611
dbaeumer merged 1 commit intomicrosoft:masterfrom
giltho:specified-logtrace

Conversation

@giltho
Copy link
Contributor

@giltho giltho commented Apr 10, 2020

@dbaeumer
Copy link
Member

@giltho should we for consistency change this one as well?

https://github.com/microsoft/vscode-languageserver-node/blob/master/jsonrpc/src/main.ts#L262

@giltho
Copy link
Contributor Author

giltho commented Apr 15, 2020

Yes I think it would be better indeed. Also, I could probably add it to the protocol proposition.

@dbaeumer
Copy link
Member

That would be great!

@giltho
Copy link
Contributor Author

giltho commented Apr 16, 2020

I updated the draft here microsoft/language-server-protocol#953 :)

@dbaeumer
Copy link
Member

@giltho could you resolve the conflict please.

@dbaeumer dbaeumer added this to the 3.16 milestone Jun 23, 2020
@giltho giltho force-pushed the specified-logtrace branch from c80e52c to 98d6b40 Compare June 23, 2020 15:26
@giltho
Copy link
Contributor Author

giltho commented Jun 23, 2020

@dbaeumer done :)

@dbaeumer dbaeumer merged commit 324d765 into microsoft:master Jun 26, 2020
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