Enclose Option to Uglify about gulp-uglify HOT 8 CLOSED

Previously I've directed users to the gulp-header and gulp-footer plugins, but they haven't seen love lately. :(

I think I'll just write a generic IIFE stream transform and suggest that.

from gulp-uglify.

Gulp Wrap ended up being what I used.

.pipe(concat('main.js'))
.pipe(wrap('(function(a){<%= contents %>}(angular));'))

from gulp-uglify.

Awesome. figures someone else already made what I was thinking about.

from gulp-uglify.

I think maybe this should be re-opend.

Uglify-js has this option and it should be used instead of having to relay on gulp-wrap. gulp-uglifyjs has this feature and accomplish this by doing it the hard way

wrap/exportAll/enclose is something that I would like to use

from gulp-uglify.

+1 for not having to rely on gulp-wrap

from gulp-uglify.

This isn't going to happen until it's supported as an API option to UglifyJS.

from gulp-uglify.

It would be helpful if this limitation (and workarounds like gulp-wrap) was documented in the readme.

from gulp-uglify.

I agree, expect more awesome docs shortly.
On Jun 9, 2015 12:59 PM, "Zach Lym" [email protected] wrote:

It would be helpful if this limitation (and workarounds like gulp-wrap)
was documented in the readme.

—
Reply to this email directly or view it on GitHub
#32 (comment)
.

from gulp-uglify.