Remove JSHint and JSCS support

Author: Turbo87

lib/config/build-config.js

@@ -1,16 +1,14 @@
 'use strict';
 
-// To create fast production builds (without minification, JSCS, or JSHint)
+// To create fast production builds (without minification and ESLint)
 // run the following:
 //
-// DISABLE_JSHINT=true DISABLE_JSCS=true DISABLE_MIN=true ember serve --environment=production
+// DISABLE_ESLINT=true DISABLE_MIN=true ember serve --environment=production
 
 var env           = process.env.EMBER_ENV || 'development';
 var isDevelopment = env === 'development';
 
 var disableESLint = !!process.env.DISABLE_ESLINT || false;
-var disableJSHint = !!process.env.DISABLE_JSHINT || false;
-var disableJSCS   = !!process.env.DISABLE_JSCS || false;
 var disableMin    = !!process.env.DISABLE_MIN || false;
 var enableDocs    = !!process.env.ENABLE_DOCS || false;
 
@@ -30,8 +28,6 @@ if (!isDevelopment && !disableMin) {
 module.exports = {
   isDevelopment:       isDevelopment,
   disableESLint:       disableESLint,
-  disableJSHint:       disableJSHint,
-  disableJSCS:         disableJSCS,
   disableMin:          disableMin,
   enableDocs:          enableDocs,
   enableTreeDebugging: enableTreeDebugging,

lib/get-es6-package.js

@@ -5,24 +5,19 @@ module.exports = getES6Package;
 
 var Funnel     = require('broccoli-funnel');
 var mergeTrees = require('broccoli-merge-trees');
-var jshintTree = require('broccoli-jshint');
 var ESLintTree = require('broccoli-lint-eslint');
-var jscsTree   = require('./jscs-tree');
 
 var config                    = require('./config/build-config');
 var packageDependencyTree     = require('./get-package-dependency-tree');
 var htmlbarsTemplatePrecompiler = require('./utils/htmlbars-template-precompiler');
 var glimmerTemplatePrecompiler = require('./utils/glimmer-template-precompiler');
 
 var disableESLint = config.disableESLint;
-var disableJSHint = config.disableJSHint;
-var disableJSCS   = config.disableJSCS;
 
 function getES6Package(packages, packageName, opts) {
   var pkg        = packages[packageName];
   var options    = opts || {};
   var jsRegExp   = /js$/;
-  var configPath = options.configPath || '';
 
   var libTree;
 
@@ -94,12 +89,6 @@ function getES6Package(packages, packageName, opts) {
     },
   });
 
-  var libJSCSTree = jscsTree(libTree, {
-    configPath: configPath
-  });
-
-  var libJSHintTree = jshintTree(libTree);
-
   var libESLintTree = new ESLintTree(libTree, { testGenerator: 'qunit' });
 
   if (pkg.hasTemplates) {
@@ -125,10 +114,6 @@ function getES6Package(packages, packageName, opts) {
     destDir: '/' + packageName + '/tests'
   });
 
-  var testJSCSTree   = jscsTree(testTree, {
-    configPath: configPath
-  });
-  var testJSHintTree = jshintTree(testTree);
   var testESLintTree = new ESLintTree(testTree, { testGenerator: 'qunit' });
 
   /*
@@ -140,16 +125,6 @@ function getES6Package(packages, packageName, opts) {
     testTrees.push(libESLintTree);
     testTrees.push(testESLintTree);
   }
-
-  if (!disableJSHint) {
-    testTrees.push(libJSHintTree);
-    testTrees.push(testJSHintTree);
-  }
-
-  if (!disableJSCS) {
-    testTrees.push(libJSCSTree);
-    testTrees.push(testJSCSTree);
-  }
   testTrees.push(testTree);
 
   testTrees = testTrees.length > 0 ? mergeTrees(testTrees, { overwrite: true }) : testTree;

package.json

@@ -39,8 +39,6 @@
     "broccoli-concat": "^2.0.0",
     "broccoli-file-creator": "^1.0.0",
     "broccoli-funnel": "^1.0.1",
-    "broccoli-jscs": "^1.4.1",
-    "broccoli-jshint": "^1.1.0",
     "broccoli-kitchen-sink-helpers": "^0.2.6",
     "broccoli-lint-eslint": "^2.3.0",
     "broccoli-merge-trees": "^1.1.1",

tests/config/build-config-test.js

@@ -9,8 +9,6 @@ describe('build config', function() {
     expect(buildConfig).to.deep.equal({
       isDevelopment:       true,
       disableESLint:       false,
-      disableJSHint:       false,
-      disableJSCS:         false,
       disableMin:          false,
       enableDocs:          false,
       enableTreeDebugging: false,