Skip to content

Conversation

@rbuckton
Copy link
Contributor

@rbuckton rbuckton commented Aug 6, 2020

Adds importName properties for the class private field get/set helpers so that they work correctly with --importHelpers.

Fixes #36841

@typescript-bot typescript-bot added Author: Team For Milestone Bug PRs that fix a bug with a specific milestone labels Aug 6, 2020
@rbuckton
Copy link
Contributor Author

rbuckton commented Aug 6, 2020

Suggestions aren't working again.

/CC @weswigham, @sandersn for review

@DanielRosenwasser this might be worth taking for 4.0.

@DanielRosenwasser
Copy link
Member

@typescript-bot cherry-pick this to release-4.0

@typescript-bot
Copy link
Collaborator

typescript-bot commented Aug 6, 2020

Heya @DanielRosenwasser, I've started to run the task to cherry-pick this into release-4.0 on this PR at f53db98. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

Hey @DanielRosenwasser, I've opened #39933 for you.

typescript-bot pushed a commit to typescript-bot/TypeScript that referenced this pull request Aug 6, 2020
Component commits:
f53db98 Add missing 'importName' for private field helpers
DanielRosenwasser pushed a commit that referenced this pull request Aug 6, 2020
Component commits:
f53db98 Add missing 'importName' for private field helpers

Co-authored-by: Ron Buckton <rbuckton@microsoft.com>
@rbuckton rbuckton merged commit fca9f45 into master Aug 6, 2020
@rbuckton rbuckton deleted the fixPrivateFieldImportHelpers branch August 6, 2020 20:27
@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

Author: Team For Milestone Bug PRs that fix a bug with a specific milestone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fail to import classPrivateFieldGet from tslib when "importHelpers": true

4 participants