Skip to content
This repository has been archived by the owner on Apr 23, 2019. It is now read-only.

New dependency name: uglify-es => terser #1 #146

Open
fabiosantoscode opened this issue May 17, 2018 · 2 comments
Open

New dependency name: uglify-es => terser #1 #146

fabiosantoscode opened this issue May 17, 2018 · 2 comments

Comments

@fabiosantoscode
Copy link

There has been development on a new fork of uglify-es, which is now called terser and has its own package!

"terser": "^3.7.3"

Please upgrade to our new version.

@irae
Copy link
Collaborator

irae commented May 17, 2018

Thanks for opening the issue!

Mendel uses plugins, like glup, webpack and others.
I'll think about adding a plugin for that. But it would be even better if we could get this as contribution. For now, I'll just post a quick initial research on Terser.

For context, we use this for mendel-transform-uglify:

We use uglify only here: https://github.com/yahoo/mendel/blob/master/packages/mendel-transform-uglify/ -- The other packages using uglify are deprecated, we will move all to plugins eventually. Just keeping retro compatibility at the moment.

I found terser here:

It has no homepage that I could find, and would be nice to have a feature comparison with uglify-es, as I can't find a reason to switch right away. It still links to http://lisperator.net/uglifyjs/ which has no terser compassion table.

@fabiosantoscode do you mind going in details of terser improvements over uglify-es?

@fabiosantoscode
Copy link
Author

fabiosantoscode commented May 18, 2018

The only difference between terser and uglify-es is active development. We're not changing anything besides that. I'll add the homepage field to the package.json as you suggest. But I haven't the time to contribute the changes, sadly.

Edit:
interesting, the "homepage" fiels is there and is a valid link, but npm isn't resolving it

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

No branches or pull requests

2 participants