From 69411b7d8335562cc8982614a65445f1e9d11810 Mon Sep 17 00:00:00 2001 From: vanheeswijck Date: Wed, 27 May 2015 16:11:41 +0200 Subject: [PATCH] Issue with gulp-if / replaced with merge There is an issue with gulp-if (https://github.com/robrich/gulp-if/issues/43) and it popped up in this project. So the best solution is replace the gulp-if with a clone/merge (already used in the project). --- shared/js/dev-tasks/build/buildSteps.js | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/shared/js/dev-tasks/build/buildSteps.js b/shared/js/dev-tasks/build/buildSteps.js index d413738a..1c27e161 100644 --- a/shared/js/dev-tasks/build/buildSteps.js +++ b/shared/js/dev-tasks/build/buildSteps.js @@ -29,6 +29,10 @@ var buffer = require('vinyl-buffer'); var chalk = require('chalk'); var helper = require('../helper'); +var merge = require('event-stream').merge; +var cloner = require('../cloner'); +var doClone = cloner(); + // make it easier to get to plugins var $ = helper.$; @@ -126,8 +130,14 @@ module.exports = { file.base = file.base.replace(/[\/\\]src/, ''); }); - var result = stream.pipe($.if('**/*.scss', sassPipe())); - return result; + var sassStream = stream.pipe(doClone); + var srcStream = doClone.cloned; + sassStream = sassStream.pipe($.if('**/*.scss', sassPipe())); + + //REMARK: removed gulp-if and replaced it by merge (to avoid loosing random files in pipe) + // https://github.com/robrich/gulp-if/issues/43 + stream = merge(srcStream,sassStream); + return stream; }, embedLr: function (stream, port) {