-
-
Notifications
You must be signed in to change notification settings - Fork 167
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
Node 8 (async/await) #83
Comments
Very much agree |
Definitely a breaking change though. |
Maybe we could use |
It is best to be able to automatically sniff using a different configuration. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The backpack Babel preset does a bunch of stuff to make
async/await
supported in environments that don't support it:backpack/packages/babel-preset-backpack/index.js
Lines 24 to 41 in 5fa5008
The issue is that as of the Node 8 LTS release it's totally unnecessary since it supports
async/await
out of the box! To get it to build with vanillaasync/await
(not transpiled to generators) I had to opt-out of the shared Babel config and basically just run with this.babelrc
:It'd be A+++ super awesome if backpack did the right thing™️ for Node 8 by default, i.e. didn't transpile generators at all and didn't include
regenerator
in the bundle. 😊The text was updated successfully, but these errors were encountered: