swap jshint for eslint

Author: David Bradshaw

.eslintrc

@@ -9,6 +9,7 @@
   "parserOptions": { "ecmaVersion": 2015 },
   "rules": {
     "func-names": 0,
+    "global-require": 0,
     "no-param-reassign": 0,
     "no-plusplus": 0,
     "no-restricted-globals": 0,

gruntfile.js

@@ -1,11 +1,10 @@
-/*global module:false*/
 module.exports = function(grunt) {
   // show elapsed time at the end
-  require('time-grunt')(grunt)
+  require('time-grunt')(grunt) // eslint-disable-line import/no-extraneous-dependencies
 
   // load all grunt tasks
-  //require('load-grunt-tasks')(grunt);
-  require('jit-grunt')(grunt, {
+  // require('load-grunt-tasks')(grunt);
+  require('jit-grunt')(grunt, { // eslint-disable-line import/no-extraneous-dependencies
     'bump-only': 'grunt-bump',
     'bump-commit': 'grunt-bump',
     coveralls: 'grunt-karma-coveralls'
@@ -79,25 +78,6 @@ module.exports = function(grunt) {
       }
     },
 
-    jshint: {
-      options: {
-        asi: true,
-        eqeqeq: true,
-        laxbreak: true,
-        globals: {
-          jQuery: false,
-          require: true,
-          process: true
-        }
-      },
-      gruntfile: {
-        src: 'gruntfile.js'
-      },
-      code: {
-        src: 'src/**/*.js'
-      }
-    },
-
     uglify: {
       options: {
         sourceMap: true,
@@ -196,13 +176,13 @@ module.exports = function(grunt) {
     },
 
     eslint: {
-      target: ['src/**']
+      target: ['src/**', '*.js']
     }
   })
 
   grunt.registerTask('default', ['notest', 'karma:single'])
   grunt.registerTask('build', ['removeBlock', 'copy', 'uglify'])
-  grunt.registerTask('notest', ['eslint', 'jsonlint', 'jshint', 'build'])
+  grunt.registerTask('notest', ['eslint', 'jsonlint', 'build'])
   grunt.registerTask('test', ['clean', 'eslint', 'karma:single', 'qunit'])
   grunt.registerTask('test-watch', ['clean', 'karma:watch'])
   grunt.registerTask('travis', [

index.js

@@ -1,3 +1 @@
-'use strict'
-
 module.exports = require('./js')

karma.conf.js

@@ -71,7 +71,7 @@ module.exports = function(config) {
 
     // start these browsers
     // available browser launchers: https://npmjs.org/browse/keyword/karma-launcher
-    browsers: ['PhantomJS'], //'Chrome', 'Firefox', 'Safari', 'PhantomJS'
+    browsers: ['PhantomJS'], // 'Chrome', 'Firefox', 'Safari', 'PhantomJS'
 
     // Continuous Integration mode
     // if true, Karma captures browsers, runs the tests and exits

package-lock.json

@@ -13,8 +13,8 @@
   ],
   "scripts": {
     "build": "npm install && grunt",
-    "eslint": "eslint src/* --color",
-    "eslint:fix": "eslint src/* --color --fix",
+    "eslint": "eslint *.js src/* --color",
+    "eslint:fix": "npm run eslint -- --fix",
     "test-watch": "grunt test-watch",
     "test": "grunt travis"
   },
@@ -35,7 +35,7 @@
     "eslint": "^6.6.0",
     "eslint-config-adjunct": "3.0.0",
     "eslint-config-airbnb-base": "^14.0.0",
-    "eslint-config-prettier": "^6.5.0",
+    "eslint-config-prettier": "^6.6.0",
     "eslint-plugin-array-func": "^3.1.3",
     "eslint-plugin-eslint-comments": "^3.1.2",
     "eslint-plugin-html": "^6.0.0",

package.json

@@ -3,7 +3,7 @@ var allTestFiles = []
 var TEST_REGEXP = /(spec|test)\.js$/i
 
 // Get a list of all the test files to include
-Object.keys(window.__karma__.files).forEach(function(file) {
+Object.keys(window.__karma__.files).forEach(function(file) { // eslint-disable-line no-underscore-dangle
   if (TEST_REGEXP.test(file)) {
     // Normalize paths to RequireJS module names.
     // If you require sub-dependencies of test files to be loaded as-is (requiring file extension)
@@ -29,5 +29,5 @@ require.config({
   deps: allTestFiles,
 
   // we have to kickoff jasmine, as it is asynchronous
-  callback: window.__karma__.start
+  callback: window.__karma__.start // eslint-disable-line no-underscore-dangle
 })