Migrate from JSHint to ESLint (as jQuery Core has)

The "jquery" ESLint config is more strict than what we had here in JSHint
so some linting violations had to be fixed as well.

Author: mgol

.eslintrc.json

@@ -0,0 +1,7 @@
+{
+	"extends": "eslint-config-jquery",
+	"root": true,
+	"env": {
+		"node": true
+	}
+}

.jshintrc

@@ -19,21 +19,23 @@ module.exports = function( grunt ) {
 						to: "<%= grunt.template.today('yyyy') %>"
 					}
 				]
-			},
+			}
 		},
 		qunit: {
 			all: [ "test/**/*.html" ]
 		},
-		jshint: {
+		eslint: {
+			options: {
+
+				// See https://github.com/sindresorhus/grunt-eslint/issues/119
+				quiet: true
+			},
 			all: {
 				src: [ "Gruntfile.js", "src/**/*.js", "test/**/*.js" ]
-			},
-			options: {
-				jshintrc: true
 			}
 		},
 		watch: {
-			files: [ "<%= jshint.all.src %>" ],
+			files: [ "<%= eslint.all.src %>" ],
 			tasks: [ "lint", "qunit" ]
 		},
 		uglify: {
@@ -55,7 +57,7 @@ module.exports = function( grunt ) {
 				banner: "/*! jQuery requestAnimationFrame - <%= pkg.version %> - " +
 					"<%= grunt.template.today('yyyy-mm-dd') %>\n" +
 					"<%= pkg.homepage ? '* ' + pkg.homepage + '\\n' : '' %>" +
-					 " * Copyright (c) <%= grunt.template.today('yyyy') %> " +
+					" * Copyright (c) <%= grunt.template.today('yyyy') %> " +
 					"<%= pkg.author.name %>; Licensed <%= pkg.license %> */"
 			}
 		}
@@ -65,6 +67,6 @@ module.exports = function( grunt ) {
 	require( "load-grunt-tasks" )( grunt );
 
 	// Default task.
-	grunt.registerTask( "default", [ "jshint", "qunit", "replace", "uglify" ] );
+	grunt.registerTask( "default", [ "eslint", "qunit", "replace", "uglify" ] );
 
 };

.jshintrc-common

@@ -7,7 +7,7 @@
  * Licensed under the MIT license.
  */
  // UMD factory https://github.com/umdjs/umd/blob/master/jqueryPlugin.js
-( function ( factory ) {
+( function( factory ) {
 	if ( typeof define === "function" && define.amd ) {
 
 		// AMD. Register as an anonymous module.
@@ -17,7 +17,7 @@
 		// Browser globals
 		factory( jQuery );
 	}
-}( function ( jQuery ) {
+} )( function( jQuery ) {
 
 if ( Number( jQuery.fn.jquery.split( "." )[ 0 ] ) >= 3 ) {
 	if ( window.console && window.console.warn ) {
@@ -49,4 +49,4 @@ if ( window.requestAnimationFrame ) {
 	};
 }
 
-} ) );
+} );

Gruntfile.js

@@ -32,11 +32,12 @@
   "license": "MIT",
   "dependencies": {},
   "devDependencies": {
+    "eslint-config-jquery": "1.0.0",
     "grunt": "1.0.1",
     "grunt-cli": "1.2.0",
-    "grunt-contrib-jshint": "1.0.0",
     "grunt-contrib-qunit": "1.2.0",
     "grunt-contrib-uglify": "2.0.0",
+    "grunt-eslint": "19.0.0",
     "grunt-text-replace": "0.4.0",
     "jquery": "1.11.3",
     "load-grunt-tasks": "3.5.2",

dist/jquery.requestanimationframe.js

@@ -1,6 +1,8 @@
 {
-	"extends": "../.jshintrc-common",
-
+	"env": {
+		"browser": true,
+		"node": false
+	},
 	"globals": {
 		"window": true,
 		"jQuery": true,

dist/jquery.requestanimationframe.min.map

@@ -7,7 +7,7 @@
  * Licensed under the MIT license.
  */
  // UMD factory https://github.com/umdjs/umd/blob/master/jqueryPlugin.js
-( function ( factory ) {
+( function( factory ) {
 	if ( typeof define === "function" && define.amd ) {
 
 		// AMD. Register as an anonymous module.
@@ -17,7 +17,7 @@
 		// Browser globals
 		factory( jQuery );
 	}
-}( function ( jQuery ) {
+} )( function( jQuery ) {
 
 if ( Number( jQuery.fn.jquery.split( "." )[ 0 ] ) >= 3 ) {
 	if ( window.console && window.console.warn ) {
@@ -49,4 +49,4 @@ if ( window.requestAnimationFrame ) {
 	};
 }
 
-} ) );
+} );

package.json

@@ -1,8 +1,8 @@
 {
-	"extends": "../.jshintrc-common",
-
-	"browser": true,
-
+	"env": {
+		"browser": true,
+		"node": false
+	},
 	"globals": {
 		"jQuery": false,
 		"QUnit": false

src/.jshintrc renamed to src/.eslintrc.json

@@ -17,4 +17,4 @@
 		} );
 	} );
 
-}() );
+} )();