From 1cbd37a9209a753a11bb930af2a7f745cd08762c Mon Sep 17 00:00:00 2001 From: Chris Hill-Scott Date: Fri, 26 Feb 2016 13:19:46 +0000 Subject: [PATCH] Bring back SASS lint The problem with SASS lint that we found in https://github.com/alphagov/notifications-admin/pull/202 has been fixed (see https://github.com/sasstools/sass-lint/pull/550). This commit reverts #202, re-enabling linting of SASS files. --- gulpfile.babel.js | 9 ++++----- package.json | 1 + 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/gulpfile.babel.js b/gulpfile.babel.js index 6f2356c80..030c5eeb7 100644 --- a/gulpfile.babel.js +++ b/gulpfile.babel.js @@ -103,11 +103,10 @@ gulp.task('watchForChanges', function() { }); gulp.task('lint:sass', () => gulp -// Removed until gulp-sass-lint installs cleanly -// .src(paths.src + 'stylesheets/**/*.scss') -// .pipe(plugins.sassLint()) -// .pipe(plugins.sassLint.format(stylish)) -// .pipe(plugins.sassLint.failOnError()) + .src(paths.src + 'stylesheets/**/*.scss') + .pipe(plugins.sassLint()) + .pipe(plugins.sassLint.format(stylish)) + .pipe(plugins.sassLint.failOnError()) ); gulp.task('lint:js', () => gulp diff --git a/package.json b/package.json index a15a60448..6ed466634 100644 --- a/package.json +++ b/package.json @@ -38,6 +38,7 @@ "devDependencies": { "gulp-css-url-adjuster": "0.2.3", "gulp-jshint": "2.0.0", + "gulp-sass-lint": "1.1.1", "jshint": "2.9.1", "jshint-stylish": "2.1.0" }