generator-gulp-angular icon indicating copy to clipboard operation
generator-gulp-angular copied to clipboard

updating gulp-filter to 4.0.0 causes map and rev failure

Open Endellur opened this issue 8 years ago • 6 comments

Not really an issue with the current version, but when I try to update all the dependencies for npm and bower I am able to move everything to current except gulp-filter. When I upgrade that from 3.0.1 to 4.0.0, rev and map stop working. I found this issue that might be related:

https://github.com/sindresorhus/gulp-filter/issues/62

I am new to gulp and wasn't able to find the issue quickly so I thought I would enter a case and let those who are more skilled with gulp take a look. Thanks.

Endellur avatar Apr 26 '16 16:04 Endellur

For me it works. Which task is the one that fails? 3.0.1 -> 4.0.0 means major release corresponding to SemVer so there should be breaking changes. Can you post some errors?

piu130 avatar Apr 28 '16 07:04 piu130

No tasks are failing, but when the build finishes the files have not been revisioned, there are no maps and the files are not minified. If I change back to 3.0.1 it works fine.

Endellur avatar Apr 28 '16 20:04 Endellur

@mykbaker here's how I got it working with 4.0.0

in build.js, update the $.filter paths like this:

var htmlFilter = $.filter(path.join(conf.paths.tmp, '/serve/*.html'), {restore: true});
var jsFilter = $.filter(path.join(conf.paths.tmp, '/serve/**/*.js'), {restore: true});
var cssFilter = $.filter(path.join(conf.paths.tmp, '/serve/**/*.css'), {restore: true});

HTH, Max

mzellho avatar May 04 '16 11:05 mzellho

@MZellhofer that worked great, thanks.

Endellur avatar May 04 '16 18:05 Endellur

That's nice, could one of you make a PR with that?

Swiip avatar May 09 '16 09:05 Swiip

I would love to, but I am on windows at the moment and somehow I cannot get your tests running. I should be able to switch to linux in the evening and will try again. I anyways pushed the changes to here

mzellho avatar May 09 '16 12:05 mzellho