Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

LanguageServiceShim.getCompletionEntryDetails: Support undefined options#20810

Merged
2 commits merged into
mastermicrosoft/TypeScript:masterfrom
options-undefinedmicrosoft/TypeScript:options-undefinedCopy head branch name to clipboard
Dec 20, 2017
Merged

LanguageServiceShim.getCompletionEntryDetails: Support undefined options#20810
2 commits merged into
mastermicrosoft/TypeScript:masterfrom
options-undefinedmicrosoft/TypeScript:options-undefinedCopy head branch name to clipboard

Conversation

@ghost

@ghost ghost commented Dec 19, 2017

Copy link
Copy Markdown

Based on an email from @billti
The commit that added the JSON.parse call was 2b566b9, not 0f32f55 which only changed type declarations.

@billti

billti commented Dec 20, 2017

Copy link
Copy Markdown
Member

Thanks! Yeah, I spotted the commit difference after I sent the email. Sorry about that.

I tried this change on Dev14 and it looks good. Can you port to release-2.6 and merge there too. Thanks for the fast turnaround!

@ghost ghost merged commit a92a594 into master Dec 20, 2017
@ghost ghost deleted the options-undefined branch December 20, 2017 15:39
ghost pushed a commit that referenced this pull request Dec 20, 2017
…ons (#20810)

* LanguageServiceShim.getCompletionEntryDetails: Support undefined options

* Fix lint
ghost pushed a commit that referenced this pull request Dec 20, 2017
…ons (#20810) (#20818)

* LanguageServiceShim.getCompletionEntryDetails: Support undefined options

* Fix lint
@microsoft microsoft locked and limited conversation to collaborators Jun 21, 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.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.