-
Notifications
You must be signed in to change notification settings - Fork 5
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
[Feature Request] Replace Uglify with Terser (or provide as an option) #497
Comments
Yea, I’ve been keeping an eye on the es branch of Uglify. The issue is that there are still too many bugs for production use. Once it matures enough, I plan to include it
…Sent from my iPhone
On Oct 3, 2018, at 06:09, dwkns ***@***.***> wrote:
Feature request
Replace Uglify with Terser
Why
Uglify-es is no longer maintained. Uglifuy-js doesn't support ES6+ (its ES2015 only).
ES6+ is increasingly being supported in browsers and certainly increasingly used during website/web app development. It should have a first class mangler/compressor rather than having to rely on babel to work.
Terser retains API and CLI compatibility with uglify-es and ***@***.***
It should also allow minification before bundling which may lead to build time improvements.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
According to the current Uglify maintainer:
And in fact, it looks like the I think Terser has emerged as the clear successor and has proved itself as more than just a fork at this point. It’s also very actively maintained. Since it has the same API/CLI as Uglify, it should be a fairly invisible transition for both the app and users. 🤞 Supporting mangling/compression of ES6 code is a key part of bringing CodeKit up-to-date with the ES6 world, so it would be great to see it implemented. |
Yea, I’m just very wary of incorporating unproven tooling. Uglify-es was buggy and ES6 itself is much more complicated to properly minify. And nobody is shipping ES6 code to Browsers without transpiling yet—we’re a few years away from that, at least.
…Sent from my iPhone
On Dec 10, 2018, at 22:10, Sean Juarez ***@***.***> wrote:
According to the current Uglify maintainer:
uglify-es is no longer maintained.
And in fact, it looks like the uglify-es branch of Uglify JS has actually been deleted.
I think Terser has emerged as the clear successor and has proved itself as more than just a fork at this point. It’s also very actively maintained. Since it has the same API/CLI as Uglify, it should be a fairly invisible transition for both the app and users. 🤞
Supporting mangling/compression of ES6 code is a key part of bringing CodeKit up-to-date with the ES6 world, so it would be great to see it implemented.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Terser is now the default minification for Webpack. |
Done. CodeKit 3.8 has Terser built-in. It replaces UglifyJS. All applicable options that carry over from Uglify are automatically migrated for every project. New options are shown in the UI. Additionally, I'm forcing the |
Should be out in a few days, once I get the docs done. If you'd like you can try it now at https://codekitapp.com/binaries/beta/bundle7.zip |
Feature request
Replace Uglify with Terser
Why
Uglify-es is no longer maintained. Uglifuy-js doesn't support ES6+ (its ES2015 only).
ES6+ is increasingly being supported in browsers and certainly increasingly used during website/web app development. It should have a first class mangler/compressor rather than having to rely on babel to work.
Terser retains API and CLI compatibility with uglify-es and uglify-js@3.
It should also allow minification before bundling which may lead to build time improvements.
The text was updated successfully, but these errors were encountered: