Skip to content

Handle undefined input to firstDefined - #21300

Merged
1 commit merged into
masterfrom
firstDefined
Jan 19, 2018
Merged

Handle undefined input to firstDefined#21300
1 commit merged into
masterfrom
firstDefined

Conversation

@ghost

Copy link
Copy Markdown

The type declaration already declares this to handle undefined, just updated the body and some uses.

@ghost
ghost merged commit d46653a into masterJan 19, 2018
@ghost
ghost deleted the firstDefined branch January 19, 2018 18:10
Albert Siddhartha Slawinski (errendir) added a commit to errendir/TypeScript that referenced this pull request Jan 20, 2018
* origin/master: (134 commits)
Fix isTypeOfExpression in compiler API (microsoft#20875). (microsoft#20884)
add completion filter for function like body (microsoft#21257)
Make nonnull assertions and binding patterns apparent declared type locations (microsoft#20995)
For `{ type: "a" } | { type: "b" }`, find references for the union property (microsoft#21298)
configureNightly -> configurePrerelease
Create a 'configure-insiders' and 'publish-insiders' task.
Add createProgram on WatchCompilerHost
in goToDefinition, use array helpers and clean up code (microsoft#21304)
Support testing definition range of a reference gruop (microsoft#21302)
Handle `undefined` input to firstDefined (microsoft#21300)
Avoid spreading array (microsoft#21291)
LEGO: check in for master to temporary branch.
Accept new baselines
Add regression test
Properly handle contravariant inferences in inferReverseMappedType
Remove unused properties from interface Refactor (microsoft#21286)
LEGO: check in for master to temporary branch.
Fold newline logic into getNewLineOrDefaultFromHost
External test runner updates (microsoft#21276)
Report more detailed info during script debug failure
...
@microsoftMicrosoft (microsoft) locked and limited conversation to collaborators Jul 3, 2018
This pull request was closed.
Sign up for freeto 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.

1 participant

@weswigham