Merge pull request #364 from Financial-Times/build-fixes

fix error when no JS or Sass streams are returned
Financial-Times · Feb 23, 2016 · bcd029c · bcd029c
2 parents c07366f + c608458
commit bcd029c
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/lib/tasks/build.js b/lib/tasks/build.js
@@ -21,12 +21,18 @@ module.exports = function(gulp, config) {
 	const buildStream = merge();
 	const emitBuildStreamErrorEvent = buildStream.emit.bind(buildStream, 'error');
 	if (typeof config.watching === 'undefined' || config.watching === 'js') {
-		jsStream = module.exports.js(gulp, config)
-			.on('error', emitBuildStreamErrorEvent);
+		jsStream = module.exports.js(gulp, config);
+
+		if (typeof jsStream !== 'undefined') {
+			jsStream.on('error', emitBuildStreamErrorEvent);
+		}
 	}
 	if (typeof config.watching === 'undefined' || config.watching === 'sass') {
-		sassStream = module.exports.sass(gulp, config)
-			.on('error', emitBuildStreamErrorEvent);
+		sassStream = module.exports.sass(gulp, config);
+
+		if (typeof sassStream !== 'undefined') {
+			sassStream.on('error', emitBuildStreamErrorEvent);
+		}
 	}
 	if (jsStream && sassStream) {
 		buildStream.add(jsStream, sassStream);