-
Notifications
You must be signed in to change notification settings - Fork 2
[Proof of Concept]: build(rollup): add experimental mode to use rollup #782
base: master
Are you sure you want to change the base?
Conversation
☔ The latest upstream changes (presumably #797) made this pull request unmergeable. Please resolve the merge conflicts. |
'transform-es2015-spread', | ||
'transform-es2015-sticky-regex', | ||
'transform-es2015-unicode-regex', | ||
'transform-regenerator', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove es2015 transforms
|
||
// some utilitis | ||
'transform-inline-environment-variables', | ||
'transform-node-env-inline', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we use https://github.com/rollup/rollup-plugin-replace instead of these?
"rollup-plugin-babel": "^2.6.1", | ||
"rollup-plugin-commonjs": "^4.1.0", | ||
"rollup-plugin-node-globals": "^1.0.6", | ||
"rollup-plugin-node-resolve": "^2.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
update packages to latest
70bdf19
to
84e9c03
Compare
84e9c03
to
795b75c
Compare
e3ad2cc
to
9737337
Compare
☔ The latest upstream changes (presumably #799) made this pull request unmergeable. Please resolve the merge conflicts. |
TODO
This change is