Skip to content

Fix crash with Object.defineProperty for imported alias (--allowJs)#35198

Merged
sandersn merged 1 commit intomicrosoft:masterfrom
Knagis:fix-alias-defineproperty-35196
Nov 20, 2019
Merged

Fix crash with Object.defineProperty for imported alias (--allowJs)#35198
sandersn merged 1 commit intomicrosoft:masterfrom
Knagis:fix-alias-defineproperty-35196

Conversation

@Knagis
Copy link
Copy Markdown
Contributor

@Knagis Knagis commented Nov 19, 2019

Fixes #35196

Follows the same approach as #34650 and #34566

Please verify that:

  • There is an associated issue in the Backlog milestone (required)
  • Code is up-to-date with the master branch
  • You've successfully run gulp runtests locally
  • There are new or updated unit tests validating the change

@sandersn sandersn merged commit 17f5469 into microsoft:master Nov 20, 2019
@sandersn
Copy link
Copy Markdown
Member

@typescript-bot cherry-pick this to release-3.7 if you can

typescript-bot pushed a commit to typescript-bot/TypeScript that referenced this pull request Nov 20, 2019
Component commits:
9c0fab9 Fix crash with Object.defineProperty for imported alias (--allowJs)
Fixes microsoft#35196
@typescript-bot
Copy link
Copy Markdown
Collaborator

Hey @sandersn, I've opened #35240 for you.

sandersn pushed a commit that referenced this pull request Nov 20, 2019
Component commits:
9c0fab9 Fix crash with Object.defineProperty for imported alias (--allowJs)
Fixes #35196
@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.

--allowJs: Object.defineProperty on imported alias crashes compiler

3 participants