Skip to content
This repository was archived by the owner on Dec 29, 2022. It is now read-only.

Fixes completionItem/resolve response to match the LSP specification. #528

Merged
merged 1 commit into from
Oct 21, 2017
Merged

Fixes completionItem/resolve response to match the LSP specification. #528

merged 1 commit into from
Oct 21, 2017

Conversation

philip-alldredge
Copy link
Contributor

Adjusts the response to the completionItem/resolve message to be a single CompletionItem.

Reference:
https://github.com/Microsoft/language-server-protocol/blob/master/protocol.md#completionItem_resolve

@Xanewok
Copy link
Member

Xanewok commented Oct 21, 2017

Nice catch, thanks!
All tests passed except test::test_workspace_symbol, but that's to be expected right now since #522

@Xanewok Xanewok merged commit 93acbad into rust-lang:master Oct 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants