Skip to content

Conversation

@david-driscoll
Copy link
Member

…ged how Request Routers router requests

@david-driscoll david-driscoll requested a review from tintoy January 11, 2018 22:07
Copy link
Collaborator

@tintoy tintoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov
Copy link

codecov bot commented Jan 11, 2018

Codecov Report

Merging #62 into master will increase coverage by 0.04%.
The diff coverage is 71.42%.

Impacted file tree graph

@@ Coverage Diff @@ ## master #62 +/- ## ========================================== + Coverage 69.98% 70.03% +0.04%  ========================================== Files 211 211 Lines 2209 2199 -10 ========================================== - Hits 1546 1540 -6  + Misses 663 659 -4
Impacted Files Coverage Δ
src/JsonRpc/ReflectionRequestHandlers.cs 100% <ø> (+25%) ⬆️
src/JsonRpc/Reciever.cs 91.66% <100%> (ø) ⬆️
src/JsonRpc/RequestRouter.cs 87.5% <100%> (+4.16%) ⬆️
src/Server/LspRequestRouter.cs 69.84% <60%> (-1.81%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1158fd8...938b45d. Read the comment docs.

@david-driscoll david-driscoll merged commit 52df6f2 into master Jan 12, 2018
tintoy added a commit to tintoy/msbuild-project-tools-server that referenced this pull request Jan 15, 2018
Fixes a bug (OmniSharp/csharp-language-server-protocol#60 / OmniSharp/csharp-language-server-protocol#62) where language server failed to terminate when VS Code sent it a shutdown request.
tintoy added a commit to tintoy/msbuild-project-tools-vscode that referenced this pull request Jan 15, 2018
Fixes a bug (OmniSharp/csharp-language-server-protocol#60 / OmniSharp/csharp-language-server-protocol#62) where language server failed to terminate when VS Code sent it a shutdown request.
tintoy added a commit to tintoy/msbuild-project-tools-server that referenced this pull request Jan 16, 2018
Turns out this brings its own problems to the table because now the server won't complete initialisation when launched from VSCode. This reverts commit 9e3a444. Relates to OmniSharp/csharp-language-server-protocol#60 / OmniSharp/csharp-language-server-protocol#62
@david-driscoll david-driscoll deleted the request-router-update branch June 2, 2020 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants