New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(pkg): build for es2019 target #9328
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #9328 +/- ##
=======================================
Coverage 65.16% 65.16%
=======================================
Files 94 94
Lines 4105 4105
Branches 1124 1124
=======================================
Hits 2675 2675
Misses 1152 1152
Partials 278 278
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
Nice change |
OOC, what will be broken when using node ? |
@clarkdo cli dist (using dynamic imports) was getting broken |
Is dynamic import not transpiled ? |
Actually it transpiled to cjs by esbuild when using node while shouldn't (rollup is esm native and supports cjs with CommonJS plugin). Maybe we could use node if there was a node-es target... |
Right, so node version >= 12.20 in target should work as it fully supports dynamic import. |
OK, it seems not evanw/esbuild#1084 (comment) |
Types of changes
Description
Use
esbuild
withtaget: es2019
(usingnode*
transform to CJS which breaks rollup chunks!) to build dist files. Fixes #9321 Closes #9322Checklist: