-
-
Notifications
You must be signed in to change notification settings - Fork 179
switch to uglify-es #33
Comments
How can I use And still get the Thank you. |
@skorunka It's not possible until someone ports the plugin to the new API of Uglify. |
@bebraw I see, is there any workaround? |
@skorunka No workaround until someone has time to perform the upgrade. |
@bebraw Thank you. |
@skorunka Hmm, I've been using a similar workaround (see below) while waiting for this issue, and it seems to be working for me. Copying here in case it helps you. /* package.json */
"devDependencies": {
"uglify-js": "git://github.com/mishoo/UglifyJS2#harmony-v2.8.22",
"uglifyjs-webpack-plugin": "0.4.3", /* webpack.config.js */
const UglifyJSPlugin = require('uglifyjs-webpack-plugin');
new UglifyJSPlugin() |
That helped! I had to delete |
Why does the README tell us to use github.com/mishoo/UglifyJS2#harmony and not mention @johnspurlock's fix? |
@JemarJones PR welcome. Thanks. 👍 |
Try method from @johnspurlock, still not work.
This is my devDependencies:
|
The most recent uglifyjs-webpack-plugin version broke something. If I return back to 0.4.4 everything works again. |
I still get |
@ricricucit could you make a test case for that? |
Done in #63 |
Used the Yeoman fountain webapp generator, made a react project, ripped out all react stuff, and made the followiing webpack fix: webpack-contrib/uglifyjs-webpack-plugin#33 requires "uglifyjs-webpack-plugin": "0.4.3",
Used the Yeoman fountain webapp generator, made a react project, ripped out all react stuff, and made the followiing webpack fix: webpack-contrib/uglifyjs-webpack-plugin#33 requires "uglifyjs-webpack-plugin": "0.4.3",
Related somewhat to #32
get rid of all the "if using ES2017 install from git" etc, and switch to
uglify-es
only.The text was updated successfully, but these errors were encountered: