Closed bfolkens closed 4 years ago
As described in issues #208 and #189, it appears that the following notification isn't handled:
%{"id" => 2, "jsonrpc" => "2.0", "method" => "shutdown"}
This PR adds support for this notification, and tests to support. Added note to @JakeBecker - might want to check if apps/language_server/lib/language_server/server.ex:332 is still necessary. At least on my brief review of https://github.com/Microsoft/language-server-protocol/blob/master/versions/protocol-2-x.md#shutdown-request, it didn't look like that was part of the spec (which was apparently causing the error).
apps/language_server/lib/language_server/server.ex:332
As described in issues #208 and #189, it appears that the following notification isn't handled:
This PR adds support for this notification, and tests to support. Added note to @JakeBecker - might want to check if
apps/language_server/lib/language_server/server.ex:332
is still necessary. At least on my brief review of https://github.com/Microsoft/language-server-protocol/blob/master/versions/protocol-2-x.md#shutdown-request, it didn't look like that was part of the spec (which was apparently causing the error).