Skip to content

🤖 Pick PR #58895 (Fix global when typescript.js loade...) into release-5.5 #58896

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

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

typescript-bot
Copy link
Collaborator

This cherry-pick was triggered by a request on #58895.

Please review the diff and merge if no changes are unexpected.

@typescript-bot typescript-bot force-pushed the cherry-pick/58895/release-5.5 branch from 26cdc45 to 6ff91dd Compare June 17, 2024 21:09
@jakebailey
Copy link
Member

@typescript-bot pack this

want to verify esm.sh real quick

@typescript-bot
Copy link
Collaborator Author

typescript-bot commented Jun 17, 2024

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
pack this ✅ Started

@jakebailey
Copy link
Member

Oh that task doesn't work on release branches, oof

@jakebailey
Copy link
Member

I checked esm.sh directly; they actually more or less just use the cjs-module-lexer (ported to Go) and then extract out the exports then wrap the module in a fake CJS context, so all good.

@jakebailey jakebailey merged commit 02132e5 into release-5.5 Jun 18, 2024
27 checks passed
@jakebailey jakebailey deleted the cherry-pick/58895/release-5.5 branch June 18, 2024 00:08
@sandersn sandersn removed this from PR Backlog Apr 22, 2025
# 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