Merge pull request #202 from alphagov/remove-sass-linting

Temporarily remove SASS linting
This commit is contained in:
Adam Shimali
2016-02-26 11:54:32 +00:00
2 changed files with 6 additions and 7 deletions

View File

@@ -103,10 +103,11 @@ gulp.task('watchForChanges', function() {
}); });
gulp.task('lint:sass', () => gulp gulp.task('lint:sass', () => gulp
.src(paths.src + 'stylesheets/**/*.scss') // Removed until gulp-sass-lint installs cleanly
.pipe(plugins.sassLint()) // .src(paths.src + 'stylesheets/**/*.scss')
.pipe(plugins.sassLint.format(stylish)) // .pipe(plugins.sassLint())
.pipe(plugins.sassLint.failOnError()) // .pipe(plugins.sassLint.format(stylish))
// .pipe(plugins.sassLint.failOnError())
); );
gulp.task('lint:js', () => gulp gulp.task('lint:js', () => gulp

View File

@@ -29,9 +29,8 @@
"gulp-base64": "0.1.3", "gulp-base64": "0.1.3",
"gulp-concat": "2.6.0", "gulp-concat": "2.6.0",
"gulp-include": "2.1.0", "gulp-include": "2.1.0",
"gulp-jquery": "1.1.1",
"gulp-load-plugins": "1.1.0", "gulp-load-plugins": "1.1.0",
"gulp-sass": "2.1.1", "gulp-sass": "2.2.0",
"gulp-uglify": "1.5.1", "gulp-uglify": "1.5.1",
"jquery": "1.11.2", "jquery": "1.11.2",
"query-command-supported": "1.0.0" "query-command-supported": "1.0.0"
@@ -39,7 +38,6 @@
"devDependencies": { "devDependencies": {
"gulp-css-url-adjuster": "0.2.3", "gulp-css-url-adjuster": "0.2.3",
"gulp-jshint": "2.0.0", "gulp-jshint": "2.0.0",
"gulp-sass-lint": "1.1.1",
"jshint": "2.9.1", "jshint": "2.9.1",
"jshint-stylish": "2.1.0" "jshint-stylish": "2.1.0"
} }