Skip to content
This repository was archived by the owner on Aug 15, 2023. It is now read-only.

Setup @babel/plugin-transform-runtime properly with @babel/runtime#2

Open
rctay wants to merge 3 commits intowavefrontHQ:masterfrom
rctay:wip/rollup-runtime
Open

Setup @babel/plugin-transform-runtime properly with @babel/runtime#2
rctay wants to merge 3 commits intowavefrontHQ:masterfrom
rctay:wip/rollup-runtime

Conversation

@rctay
Copy link

@rctay rctay commented May 25, 2020

@babel/runtime is actually meant to be used with this plugin, per https://babeljs.io/docs/en/babel-plugin-transform-runtime

This gives us the nice ability of require()-ing the library as-is, ie. without passing -r regenerator-runtime as an argument to node.

rctay added 3 commits May 25, 2020 23:34
It was listed in 'dependencies', but package-lock.json shows it to have
been installed as a dev dependency.

Signed-off-by: Ray Chuan Tay <rctay89@gmail.com>
This is stated in the docs: https://babeljs.io/docs/en/babel-runtime

Signed-off-by: Ray Chuan Tay <rctay89@gmail.com>>
@babel/runtime is actually meant to be used with this plugin, per
https://babeljs.io/docs/en/babel-plugin-transform-runtime

By turning this on, we need to configure `runtimeHelpers` for rollup.

This allows us to `require()` the library as-is, ie. without passing
`-r regenerator-runtime` to node.

Signed-off-by: Ray Chuan Tay <rctay89@gmail.com>
Copy link
Contributor

@yzheqing yzheqing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc @yjw9012

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.

2 participants