Merge pull request #242 from yeoman/plumber

Use Sass built-in error logging
yeoman · Dec 15, 2014 · bb6070f · bb6070f
2 parents 4ba2680 + 7851bc0
commit bb6070f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/app/templates/_package.json b/app/templates/_package.json
@@ -19,8 +19,7 @@
     "gulp-jshint": "^1.5.3",
     "gulp-livereload": "^2.0.0",
     "gulp-load-plugins": "^0.7.1",
-    "gulp-minify-html": "^0.1.6",<% if (includeSass) { %>
-    "gulp-plumber": "^0.6.3",<% } %>
+    "gulp-minify-html": "^0.1.6",
     "gulp-postcss": "^1.0.2",<% if (includeSass) { %>
     "gulp-ruby-sass": "^0.7.1",<% } %>
     "gulp-size": "^1.1.0",

diff --git a/app/templates/gulpfile.js b/app/templates/gulpfile.js
@@ -6,11 +6,11 @@ var $ = require('gulp-load-plugins')();
 
 gulp.task('styles', function () {<% if (includeSass) { %>
   return gulp.src('app/styles/main.scss')
-    .pipe($.plumber())
     .pipe($.rubySass({
       style: 'expanded',
       precision: 10
-    }))<% } else { %>
+    }))
+    .on('error', function (err) { console.log(err.message); })<% } else { %>
   return gulp.src('app/styles/main.css')<% } %>
     .pipe($.postcss([
       require('autoprefixer-core')({browsers: ['last 1 version']})