You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Jul 29, 2026. It is now read-only.
Repository navigation
This repository was archived by the owner on Jul 29, 2026. It is now read-only.
gulp-filter@4.0.0 will miss file when building, while gulp-filter@3.0.1 OK #2146
when I use gulp build, i found app.js did not uglity or rev
problem is
gulp-filter@4.0.0 will miss some files.
if i reset to gulp-filter@3.0.1, it's OK
see the gulp file here
in gulp.task('build:client')
return gulp.src(paths.client.mainView)
.pipe(plugins.useref())
.pipe(appFilter)
.pipe(plugins.addSrc.append('.tmp/templates.js'))
.pipe(plugins.concat('app/app.js'))
.pipe(appFilter.restore) ------------- // app/app.js is included
.pipe(jsFilter) ------------- // app/app.js is excluded!
.pipe(plugins.ngAnnotate())
.pipe(plugins.uglify())
.pipe(jsFilter.restore)
.pipe(cssFilter)
.pipe(plugins.cleanCss({
processImportFrom: ['!fonts.googleapis.com']
}))
.pipe(cssFilter.restore)
.pipe(htmlBlock) //the same problem here
.pipe(plugins.rev())
.pipe(htmlBlock.restore)
.pipe(plugins.revReplace({manifest}))
.pipe(gulp.dest(${paths.dist}/${clientPath}));
Activity
Awk34 commentedon Aug 12, 2016
Could you elaborate more? Why would gulp-filter 4 break this while it works with 3?
jacinchan commentedon Aug 12, 2016
gulp-filter@4.0.0 change the way of calling multimatch
4.0.0 -- > multimatch(path.relative(file.cwd, file.path), pattern, options)
test
that the problem.
so if we use gulp-filter@4.0.0
our filter option must add {dot:true}
that's
Awk34 commentedon Aug 12, 2016
Would you be comfortable sending a pull request to the 3.x.x branch?
fix(gulpfile): gulp-filter@4.0.0 need "dot:true" option
jacinchan commentedon Aug 13, 2016
well, it's my first pull request... i don't know how to use actually.
fix(gulpfile): gulp-filter@4.0.0 need "dot:true" option