browserify-middleware icon indicating copy to clipboard operation
browserify-middleware copied to clipboard

Use uglifyify

Open ForbesLindesay opened this issue 11 years ago • 4 comments

We should try using https://npmjs.org/package/uglifyify by @hughsk whenever minify is true for slightly better compression (sometimes).

ForbesLindesay avatar Jul 29 '13 12:07 ForbesLindesay

How does closure compiler compare?

killroy42 avatar Jul 30 '14 11:07 killroy42

closure is broken: https://github.com/ForbesLindesay/umd/pull/14 UglifyJS doesn't break your code.

ForbesLindesay avatar Jul 31 '14 13:07 ForbesLindesay

My pull request regarding supporting async hooks is very generic though. I hope you'll take a look. If that pull gets accepted I would also add some more hooks. For example to be able to log out when the process has completed and has been cached.

killroy42 avatar Jul 31 '14 13:07 killroy42

OK, this is un-related to that. This is talking about using uglifyify as a transform by default. Roughly equivalent to adding:

browserify.settings.production('transform', [require('uglifyify')]);

by default.

ForbesLindesay avatar Jul 31 '14 14:07 ForbesLindesay