Skip to content

Fix conditional type variance - #22869

Merged
Anders Hejlsberg (ahejlsberg) merged 7 commits into
masterfrom
fixConditionalTypeVariance
Mar 26, 2018
Merged

Fix conditional type variance#22869
Anders Hejlsberg (ahejlsberg) merged 7 commits into
masterfrom
fixConditionalTypeVariance

Conversation

@ahejlsberg

@ahejlsberg Anders Hejlsberg (ahejlsberg) commented Mar 25, 2018

Copy link
Copy Markdown
Member

This PR adds a missing type relationship for conditional types: Two conditional types T1 extends U1 ? X1 : Y1 and T2 extends U2 ? X2 : Y2 are related if

  • one or both of T1 and T2 is related to the other,
  • U1 and U2 are identical types,
  • X1 is related to X2, and
  • Y1 is related to Y2.

Fixes #22860.

@ahejlsberg

Copy link
Copy Markdown
Member Author

Daniel Rosenwasser (@DanielRosenwasser) Mohamed Hegazy (@mhegazy) I think we should include this one in 2.8.1.

@mhegazy

Copy link
Copy Markdown
Contributor

it is a bit late for 2.8.1, but should be in 2.8.2

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