Consider uglify-es by default

[tomkel]

Webpack uglify plugin just started using uglify-es by default.

webpack-contrib/uglifyjs-webpack-plugin#63

[TrySound]

PR welcome

[paulocoghi]

@TrySound, since uglify-es represents the UglifyJS version 3, and its API is not backward compatible with version 2, it would be interesting to test it before merge (apparently the tests passed).

[TrySound]

This project uses uglify3

[paulocoghi]

Nice, thanks! In this case, is that PR apt to be approved? Or does it no longer make sense?

I'm asking this because of the warning about ES2015. Is the following approach still necessary?

import { rollup } from 'rollup';
import uglify from 'rollup-plugin-uglify';
import { minify } from 'uglify-es';

rollup({
    entry: 'main.js',
    plugins: [
        uglify({}, minify)
    ]
});

[VincentCharpentier]

Seems like ES6 is still an issue with this plugin.
Am I missing a newer plugin that should replace this one ?

The trick mentioned above is not longer possible:

uglify({}, minify) // This won't work

The uglify function takes only one options argument now

I tried @babel/preset-env with forceAllTransforms: true because the docs says:

Enable this option if you want to force running all transforms, which is useful if the output will be run through UglifyJS or an environment that only supports ES5.

But it's not working so far

[TrySound]

Yes, use this one https://github.com/TrySound/rollup-plugin-terser.

[VincentCharpentier]

Thanks for the quick reply, works really well now 👍