Skip to content

Fix bug in reduceEachChild for tagged template expressions#27964

Merged
DanielRosenwasser merged 3 commits into
microsoft:masterfrom
Igorbek:issue27854
Oct 22, 2018
Merged

Fix bug in reduceEachChild for tagged template expressions#27964
DanielRosenwasser merged 3 commits into
microsoft:masterfrom
Igorbek:issue27854

Conversation

@Igorbek
Copy link
Copy Markdown
Contributor

@Igorbek Igorbek commented Oct 18, 2018

Fixes #27854

The bug was in reduceEachChild where it was not reducing type arguments (recently introduced) of tagged template expressions.

Added a test case from the bug report.
/cc @DanielRosenwasser

@@ -0,0 +1,3 @@
newName<{
a: string;
Copy link
Copy Markdown
Contributor Author

@Igorbek Igorbek Oct 18, 2018

Choose a reason for hiding this comment

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

here I accepted the broken functionality first before fixing it

@sheetalkamat sheetalkamat requested a review from rbuckton October 19, 2018 22:21
@DanielRosenwasser DanielRosenwasser merged commit 68ce68d into microsoft:master Oct 22, 2018
weswigham added a commit to weswigham/TypeScript that referenced this pull request Oct 23, 2018
weswigham added a commit that referenced this pull request Oct 23, 2018
@Igorbek Igorbek deleted the issue27854 branch October 23, 2018 23:54
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 21, 2025
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.

4 participants