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

Contextually type parenthesized expressions #1648

Merged
merged 5 commits into from
Jan 14, 2015

Conversation

DanielRosenwasser
Copy link
Member

This PR addresses #920.

It is also a follow-up PR to #1628 because there were issues with which commits would end up in master.

~~~
!!! error TS2453: The type argument for type parameter 'T' cannot be inferred from the usage. Consider specifying the type arguments explicitly.
!!! error TS2453: Type argument candidate 'string' is not a valid type argument because it is not a supertype of candidate 'T'.
Copy link
Contributor

Choose a reason for hiding this comment

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

'T' in this message should say 'number'

@JsonFreeman
Copy link
Contributor

👍

@DanielRosenwasser DanielRosenwasser changed the title Contextual type parenthesized expressions Contextually type parenthesized expressions Jan 12, 2015
@DanielRosenwasser DanielRosenwasser added Breaking Change Would introduce errors in existing code and removed Breaking Change Would introduce errors in existing code labels Jan 12, 2015
@ahejlsberg
Copy link
Member

👍

DanielRosenwasser added a commit that referenced this pull request Jan 14, 2015
…pressions

Contextually type parenthesized expressions
@DanielRosenwasser DanielRosenwasser merged commit bfe63cc into master Jan 14, 2015
@mhegazy mhegazy deleted the contextualTypeParenthesizedExpressions branch January 15, 2015 23:19
@microsoft microsoft locked and limited conversation to collaborators Jun 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Breaking Change Would introduce errors in existing code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants