Skip to content

When contextual type doesnt find property in unit type, use the index signature from unit type(fix the incorrectly used union type to get the signature) #27952

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

Merged
merged 1 commit into from
Oct 17, 2018

Conversation

sheetalkamat
Copy link
Member

Fixes #27949

…om unit type(fix the incorrectly used union type to get the signature)

Fixes #27949
Copy link
Member

@weswigham weswigham left a comment

Choose a reason for hiding this comment

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

The CI failures look like lint changes, but seem unrelated? Know why? Did the better contextual typing somehow affect the types of the things being asserted in a positive way?

@weswigham
Copy link
Member

Build failure seems unrelated; it's happening elsewhere. I'm guessing tslint updated or something?

@sheetalkamat sheetalkamat merged commit 7b9ae4d into master Oct 17, 2018
@sheetalkamat sheetalkamat deleted the contextualSignature branch October 17, 2018 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants