search

ForbesLindesay / browserify-middleware

express middleware for browserify, done right
http://browserify.org
MIT License
381 stars 66 forks source link

Use uglifyify #13

Open ForbesLindesay opened 11 years ago

ForbesLindesay commented 11 years ago

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

killroy42 commented 10 years ago

How does closure compiler compare?

ForbesLindesay commented 10 years ago

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

killroy42 commented 10 years ago

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.

ForbesLindesay commented 10 years ago

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.