Skip to content

feat(Terser): code minification migration #709

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

Merged

Conversation

peterblazejewicz
Copy link
Contributor

This fixes webpack build in production mode
For more details, see @webpack/webpack#7908

Thanks!

ERROR in main-server.js from UglifyJs
    Unexpected token: name (NodeList) [./node_modules/domino/lib/NodeList.es6.js:4,0][main-server.js:99960,23]

@Flood
Copy link
Contributor

Flood commented Jan 9, 2019

You need to make the same changes to webpack.config.vendor.js as well

@peterblazejewicz
Copy link
Contributor Author

@Flood Good point, I'll review and asap.
Thanks!

@peterblazejewicz peterblazejewicz force-pushed the feat/migrate-minification branch from 2b4b876 to 7ff37be Compare January 9, 2019 20:09
This fixes webpack build in production mode
See @webpack/webpack#7908

Thanks!
@peterblazejewicz peterblazejewicz force-pushed the feat/migrate-minification branch from 7ff37be to 1f3d181 Compare January 9, 2019 20:11
@peterblazejewicz
Copy link
Contributor Author

@Flood
Updated and squashed with changes.
Thanks again!

@peterblazejewicz peterblazejewicz changed the title Migrate code minification to Terser plugin feat(Terser): code minification migration Jan 9, 2019
@MarkPieszak
Copy link
Member

Got latest and merging it in! You rock @peterblazejewicz 🎸🎸🎸🎸

@MarkPieszak MarkPieszak merged commit 49cfeba into TrilonIO:master Feb 10, 2019
@peterblazejewicz peterblazejewicz deleted the feat/migrate-minification branch February 10, 2019 14:04
@peterblazejewicz
Copy link
Contributor Author

YAW

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants