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
output ES5 code so that we can run the phantomjs tests in webapp #341
Conversation
Deploy preview for wonder-blocks ready! Built with commit a0565d8 |
Codecov Report
@@ Coverage Diff @@
## master #341 +/- ##
=======================================
Coverage 89.38% 89.38%
=======================================
Files 105 105
Lines 1479 1479
Branches 293 293
=======================================
Hits 1322 1322
Misses 135 135
Partials 22 22 Continue to review full report at Codecov.
|
"@babel/preset-env", | ||
{ | ||
targets: { | ||
node: true, |
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.
This allows us to use async
/await
in our test code without having to load the regenerator runtime.
Which change in here actually changes the output to ES5? I don't see it. |
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.
Looks good to me! Thanks for doing this!
This will also avoid having to update UglifyJS which also can't handle ES6 code. Upgrading both the phantomjs test runner and UglifyJS as part of the current build system aren't good uses of time given that we'll soon be replacing our current build system and test runner.