search

actum / gulp-dev-stack

Actum dev stack based on gulp
MIT License
11 stars 7 forks source link

#167 - remove gulp uglify #168

Closed Lin84 closed 7 years ago

Lin84 commented 7 years ago

remove 'gulp-uglify' from package json and gulp task for JS for minify JS will use only: 'uglifyify'

vbulant commented 7 years ago

Having gulp-uglify in the pipeline even after uglifyify transform was on purpose back when we created the pipeline. The motivation was that uglifyify optimizes each file separately before being added to the bundle while uglify should still process the whole bundle after it’s combined.

I can’t find any docs to that, but there is at least a hint: http://stackoverflow.com/a/40257609/1149395

I think we should test whether there are any noticeable savings when using both of them and keep uglify there if so.