Merge pull request #159 from Financial-Times/jshint-even-with-no-main-js

JSHint should still run even if there is no main.js
Financial-Times · Dec 16, 2014 · 954cac4 · 954cac4
2 parents 0770b7c + 3397893
commit 954cac4
Showing 1 changed file with 0 additions and 10 deletions.
diff --git a/lib/tasks/verify.js b/lib/tasks/verify.js
@@ -1,6 +1,5 @@
 'use strict';
 
-var files = require('../helpers/files');
 var log = require('../helpers/log');
 var path = require('path');
 var fs = require('fs');
@@ -62,10 +61,6 @@ function pathsToGlob(paths) {
  */
 module.exports.scssLint = function(gulp, config) {
 	config = config || {};
-	if (!config.sass && !files.getMainSassPath()) {
-		log.secondaryError('No main.scss');
-		return;
-	}
 	var configPath = config.scssLintPath || path.join(__dirname, '/../../config/scss-lint.yml');
 
 	return gulp.src(['**/*.scss'].concat(excludeFiles))
@@ -79,11 +74,6 @@ module.exports.scssLint = function(gulp, config) {
  */
 module.exports.jsHint = function(gulp, config) {
 	config = config || {};
-	if (!config.js && !files.getMainJsPath()) {
-		log.secondary('No main.js');
-		return;
-	}
-
 	var configPath = config.jsHintPath || path.join(__dirname, '/../../config/jshint.json');
 
 	return gulp.src(['**/*.js'].concat(excludeFiles))