Skip to content

Named Type Arguments & Partial Type Argument Inference #23696

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.

Sign up for GitHub

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

Closed
weswigham wants to merge 14 commits into from
Closed

Named Type Arguments & Partial Type Argument Inference #23696

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
b881226
Basic parsing+handling code that is aware of named type arguments, no…
weswigham Apr 20, 2018
a75efc4
Fix small jsx inference bug
weswigham Apr 24, 2018
e578802
A a big pile of error messages
weswigham Apr 24, 2018
65700dc
Small fix + refactor, lint fix
weswigham Apr 25, 2018
b526f5d
handle defaults
weswigham Apr 26, 2018
fc69ca4
Merge branch 'master' into named-type-arguments-beta
weswigham Apr 26, 2018
d71f989
Inline variables
weswigham Apr 26, 2018
5185c77
Handle defaults better
weswigham Apr 26, 2018
59f66dd
Update comments, simplify getSyntheticInferType
weswigham Apr 27, 2018
26ad385
Merge branch 'master' into named-type-arguments-beta
weswigham May 3, 2018
7a108ce
Error number changes form merge
weswigham May 3, 2018
63731ba
Merge commit '3fe946df78c270817192c9eb7a22e7111c8a592a' into named-ty…
weswigham May 23, 2018
1a6154a
Merge branch 'master' into named-type-arguments-beta
weswigham May 23, 2018
9d9d57e
Merge branch 'master' into named-type-arguments-beta
weswigham May 23, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'master' into named-type-arguments-beta
  • Loading branch information
@weswigham
weswigham committed Apr 26, 2018
commit fc69ca4c4da274fa89cdb85ab8ec4acd0c56fc85
3 changes: 1 addition & 2 deletions src/compiler/checker.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7300,8 +7300,7 @@ namespace ts {
}

function getConstraintDeclaration(type: TypeParameter) {
if (!type.symbol) return;
const decl = getDeclarationOfKind<TypeParameterDeclaration>(type.symbol, SyntaxKind.TypeParameter);
const decl = type.symbol && getDeclarationOfKind<TypeParameterDeclaration>(type.symbol, SyntaxKind.TypeParameter);
return decl && decl.constraint;
}

Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.