Generic defaults by RyanCavanaugh · Pull Request #6354 · microsoft/TypeScript
Navigation Menu
{{ message }}
microsoft / TypeScript Public
- Notifications You must be signed in to change notification settings
- Fork 13.3k
Closed
RyanCavanaugh wants to merge 3 commits intomicrosoft:masterfrom
Closed
Generic defaults#6354
RyanCavanaugh wants to merge 3 commits intomicrosoft:masterfrom
Generic defaults#6354
RyanCavanaugh wants to merge 3 commits intomicrosoft:masterfrom
Conversation
Copy link
Member
RyanCavanaugh
commented
Jan 5, 2016
RyanCavanaugh
commented
A mostly-complete implementation for #2175
Not done yet:
- Check that all defaults, when present, are identical
- Check that defaults are assignable to constraints
/cc @ahejlsberg
RyanCavanaugh added 3 commits
January 1, 2016 18:02
msftclas
added
the
cla-not-required
label
Copy link
Contributor
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not an error.
RyanCavanaugh
self-assigned this
mhegazy
mentioned this pull request
Copy link
Contributor
mhegazy
commented
Sep 13, 2016
mhegazy commented
Sep 13, 2016@RyanCavanaugh can you please refresh this PR?
| let a4: HasDefault2; | ||
|
|
||
| /** Required may not follow optional */ | ||
| interface Wrong<T, U = number> { } |
Copy link
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you mean interface Wrong<T = number, U> {}?
rbuckton
mentioned this pull request
Copy link
Member Author
RyanCavanaugh
commented
Feb 13, 2017
RyanCavanaugh commented
Feb 13, 2017Superceded by #13487
RyanCavanaugh
closed this
microsoft
locked and limited conversation to collaborators
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.