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

Support multiple completions with the same name but different source … #19496

Merged
1 commit merged into from
Oct 26, 2017

Conversation

ghost
Copy link

@ghost ghost commented Oct 26, 2017

Port of #19455

…module (#19455)

* Support multiple completions with the same name but different source module

* Use optional parameters for source

* Simplify use of `uniques`

* Update test

* Fix `undefined` error
@ghost
Copy link
Author

ghost commented Oct 26, 2017

@mhegazy I want to get #19455 (comment) done before publishing a new release.

@mhegazy
Copy link
Contributor

mhegazy commented Oct 26, 2017

i will wait on the new change for the release, but i want to kick of a build for testing and want this change to be in it.

@ghost ghost merged commit a7e172b into release-2.6 Oct 26, 2017
@ghost ghost deleted the release-2.6_multiple_import_completions branch October 26, 2017 21:03
@microsoft microsoft locked and limited conversation to collaborators Jun 14, 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.

None yet

1 participant