Skip to content

Update to @rollup/plugin-babel#30742

Merged
XhmikosR merged 1 commit intomasterfrom
master-xmr-rollup-babel
May 26, 2020
Merged

Update to @rollup/plugin-babel#30742
XhmikosR merged 1 commit intomasterfrom
master-xmr-rollup-babel

Conversation

@XhmikosR
Copy link
Copy Markdown
Member

@XhmikosR XhmikosR commented May 5, 2020

Fixes #30683

Changelog: https://github.com/rollup/plugins/blob/master/packages/babel/CHANGELOG.md#500

Removed externalHelpers & runtimeHelpers options. There is now a single babelHelpers option which can take one of 'bundled', 'inline', 'runtime' and 'external' as a value. The default is 'bundled' which matches 4.x behavior, but it is recommended to configure this option explicitly.

So, apart from changing the import, I also had to adapt to the babelHelpers helper option.

Also, I removed the plugins option in js/tests/karma.conf.js since we already have this in our .babelrc.js in root, but we need to confirm if it's being discovered automatically.

TODO:

  • Verify all builds are the same
  • Verify that .babelrc.js is discovered and used by karma
  • Backport this in v4-dev

@XhmikosR XhmikosR requested a review from a team as a code owner May 5, 2020 16:10
@XhmikosR XhmikosR force-pushed the master-xmr-rollup-babel branch 5 times, most recently from 6a0ca51 to f90273d Compare May 8, 2020 18:10
@XhmikosR XhmikosR force-pushed the master-xmr-rollup-babel branch 2 times, most recently from d5bca7f to 154455b Compare May 14, 2020 07:45
@XhmikosR
Copy link
Copy Markdown
Member Author

Not sure who can review this, it does seem to work

* Verify that `.babelrc.js` is discovered and used by karma

This one I'm not sure about, but the dist builds are the same.

@XhmikosR XhmikosR force-pushed the master-xmr-rollup-babel branch from d35e08b to 6cfb79d Compare May 26, 2020 02:05
@XhmikosR
Copy link
Copy Markdown
Member Author

@mdo I need to proceed someway here...

@XhmikosR XhmikosR merged commit 340236c into master May 26, 2020
@XhmikosR XhmikosR deleted the master-xmr-rollup-babel branch May 26, 2020 03:14
XhmikosR added a commit that referenced this pull request May 26, 2020
Update to `@rollup/plugin-babel`
XhmikosR added a commit that referenced this pull request May 29, 2020
Update to `@rollup/plugin-babel`
XhmikosR added a commit that referenced this pull request May 30, 2020
Update to `@rollup/plugin-babel`
XhmikosR added a commit that referenced this pull request Jun 4, 2020
Update to `@rollup/plugin-babel`
XhmikosR added a commit that referenced this pull request Jun 4, 2020
Update to `@rollup/plugin-babel`
XhmikosR added a commit that referenced this pull request Jun 4, 2020
Update to `@rollup/plugin-babel`
olsza pushed a commit to olsza/bootstrap that referenced this pull request Oct 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update rollup-plugin-babel to @rollup/plugin-babel

2 participants