Skip to content

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

Merged
Nathan Shively-Sanders (sandersn) merged 1 commit into
microsoft:masterfrom
Knagis:fix-alias-defineproperty-35196
Nov 20, 2019
Merged

Nathan Shively-Sanders (sandersn) merged 1 commit into
microsoft:masterfrom
Knagis:fix-alias-defineproperty-35196

Conversation

@Knagis

Copy link
Copy Markdown
Contributor

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
Nathan Shively-Sanders (sandersn) merged commit 17f5469 into microsoft:master Nov 20, 2019
@sandersn

Copy link
Copy Markdown
Member

TypeScript Bot (@typescript-bot) cherry-pick this to release-3.7 if you can

TypeScript Bot (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
Contributor

Hey Nathan Shively-Sanders (@sandersn), I've opened #35240 for you.

Nathan Shively-Sanders (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 (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