Skip to content

Add CompletionInfo command to protocol#25080

Merged
2 commits merged intomasterfrom
completion_info
Jun 20, 2018
Merged

Add CompletionInfo command to protocol#25080
2 commits merged intomasterfrom
completion_info

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jun 19, 2018

Fixes #17488

@ghost ghost requested a review from mjbvz June 19, 2018 22:04
Comment thread src/server/protocol.ts
body?: CompletionInfo;
}

export interface CompletionInfo {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

i would add a comment on this and on Completions to state the difference and ask API users to use the new command moving forward.

@ghost ghost merged commit 1c3259d into master Jun 20, 2018
@ghost ghost deleted the completion_info branch June 20, 2018 16:13
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 21, 2025
This pull request was closed.
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