Skip to content

🤖 Pick PR #48775 (Fix handling of prologue statements...) into release-4.6 #48781

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Conversation

typescript-bot
Copy link
Collaborator

This cherry-pick was triggered by a request on #48775
Please review the diff and merge if no changes are unexpected.
You can view the cherry-pick log here.

cc @jakebailey

Component commits:
74c337b Fix handling of prologue statements when there are parameter property declarations If there is a prologue (and no super) in a constructor, the prologue won't be skipped when calculating the parameter properties, leading to the prologue being included twice

bd2a2ec Add second test case

d45098e Update to match old TS version: parameter properties after prologue

891cb6f Parametert properties should be added after any prologue
@jakebailey
Copy link
Member

This unfortunately needs #48782 as the ts.ts transform affects these tests. Maybe it's better to just pick all of these in one PR? (But, then maybe squash merging isn't a good idea.)

@jakebailey
Copy link
Member

@DanielRosenwasser did you mean to merge this one in with the other two?

@DanielRosenwasser
Copy link
Member

Yup, I had to wait for CI to pass first, thank you!

@DanielRosenwasser DanielRosenwasser merged commit 774f419 into microsoft:release-4.6 May 5, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
For Uncommitted Bug PR for untriaged, rejected, closed or missing bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants