Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bug: Support this. completions even when isGlobalCompletion is false #21330

Merged
1 commit merged into from
Jan 25, 2018

Conversation

ghost
Copy link

@ghost ghost commented Jan 22, 2018

Fixes #21320

Does anyone know why we return isGlobalCompletion as part of a CompletionInfo and why it is not true in an expression context?

@ghost ghost requested review from DanielRosenwasser, armanio123 and mhegazy January 22, 2018 16:15
@sheetalkamat
Copy link
Member

#11338 added isGlobalCompletion

@mhegazy
Copy link
Contributor

mhegazy commented Jan 24, 2018

We also need to move this to release-2.7

@ghost ghost merged commit 0b7f6d5 into master Jan 25, 2018
@ghost ghost deleted the completionkindnone branch January 25, 2018 15:42
ghost pushed a commit that referenced this pull request Jan 25, 2018
ghost pushed a commit that referenced this pull request Jan 25, 2018
@microsoft microsoft locked and limited conversation to collaborators Jul 3, 2018
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