From c33108cad16ba6ff831d5cbc45b9dfbf12650e0e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Go=C5=82=C4=99biowski?= <m.goleb@gmail.com>
Date: Thu, 11 Jul 2013 12:52:07 -0400
Subject: [PATCH] Update devDependencies, adjust .jshintrc to jsHint 2.x, use
 .jshintignore. Close gh-1283.

---
 .jshintignore      |  7 +++++++
 Gruntfile.js       | 12 ++++--------
 package.json       |  8 ++++----
 src/.jshintrc      |  1 -
 test/.jshintignore |  3 ---
 test/.jshintrc     | 47 +++++++++++++++++++++++-----------------------
 6 files changed, 39 insertions(+), 39 deletions(-)
 create mode 100644 .jshintignore
 delete mode 100644 test/.jshintignore

diff --git a/.jshintignore b/.jshintignore
new file mode 100644
index 000000000..6c1c77037
--- /dev/null
+++ b/.jshintignore
@@ -0,0 +1,7 @@
+test/qunit/**
+test/data/jquery-1.9.1.ajax_xhr.min.js
+test/data/badcall.js
+test/data/badjson.js
+test/data/json_obj.js
+test/data/readywaitasset.js
+test/data/readywaitloader.js
diff --git a/Gruntfile.js b/Gruntfile.js
index 0e782ff4b..adc3b7384 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -15,7 +15,8 @@ module.exports = function( grunt ) {
 			} catch(e) {}
 			return data;
 		},
-		srcHintOptions = readOptionalJSON("src/.jshintrc");
+		fs = require( "fs" ),
+		srcHintOptions = readOptionalJSON( "src/.jshintrc" );
 
 	// The concatenated file won't pass onevar
 	// But our modules can
@@ -85,9 +86,7 @@ module.exports = function( grunt ) {
 				}
 			},
 			tests: {
-				// TODO: Once .jshintignore is supported, use that instead.
-				// issue located here: https://github.com/gruntjs/grunt-contrib-jshint/issues/1
-				src: [ "test/data/{test,testinit,testrunner}.js", "test/unit/**/*.js" ],
+				src: [ "test/**/*.js" ],
 				options: {
 					jshintrc: "test/.jshintrc"
 				}
@@ -453,7 +452,7 @@ module.exports = function( grunt ) {
 
 	// Process files for distribution
 	grunt.registerTask( "dist", function() {
-		var stored, flags, paths, fs, nonascii;
+		var stored, flags, paths, nonascii;
 
 		// Check for stored destination paths
 		// ( set in dist/.destination.json )
@@ -468,7 +467,6 @@ module.exports = function( grunt ) {
 		});
 
 		// Ensure the dist files are pure ASCII
-		fs = require( "fs" );
 		nonascii = false;
 
 		distpaths.forEach(function( filename ) {
@@ -559,8 +557,6 @@ module.exports = function( grunt ) {
 	// The problem is caused by the pre-uglify task.
 	// Also, remove temporary files.
 	grunt.registerMultiTask( "post-uglify", function() {
-		var fs = require( "fs" );
-
 		this.files.forEach(function( mapping ) {
 			var mapFileName = mapping.src[ 0 ];
 
diff --git a/package.json b/package.json
index 2ae795396..b98eb051b 100644
--- a/package.json
+++ b/package.json
@@ -23,15 +23,15 @@
 	],
 	"dependencies": {},
 	"devDependencies": {
-		"grunt": "0.4.1",
+		"grunt": "~0.4.1",
 		"grunt-compare-size": "~0.4.0",
-		"grunt-git-authors": "1.2.0",
+		"grunt-git-authors": "~1.2.0",
 		"grunt-contrib-watch": "~0.4.4",
 		"grunt-contrib-jshint": "~0.6.0",
 		"grunt-contrib-uglify": "~0.2.2",
-		"gzip-js": "0.3.1",
+		"gzip-js": "0.3.2",
 		"testswarm": "~1.1.0",
-		"archiver": "~0.4.2"
+		"archiver": "~0.4.6"
 	},
 	"keywords": []
 }
diff --git a/src/.jshintrc b/src/.jshintrc
index c61a8325d..ba16cd5db 100644
--- a/src/.jshintrc
+++ b/src/.jshintrc
@@ -16,7 +16,6 @@
 	"sub": true,
 
 	"browser": true,
-	"es5": true,
 
 	"globals": {
 		"jQuery": true,
diff --git a/test/.jshintignore b/test/.jshintignore
deleted file mode 100644
index 8f2ed18cf..000000000
--- a/test/.jshintignore
+++ /dev/null
@@ -1,3 +0,0 @@
-qunit/
-data/badjson.js
-data/jquery-1.9.1.ajax_xhr.min.js
diff --git a/test/.jshintrc b/test/.jshintrc
index f92ff88cd..75c50eeeb 100644
--- a/test/.jshintrc
+++ b/test/.jshintrc
@@ -21,32 +21,33 @@
 	"wsh": true,
 
 	"globals": {
-		"DOMParser": true,
+		"DOMParser": false,
+		"QUnit": false,
+		"ok": false,
+		"equal": false,
+		"asyncTest": false,
+		"notEqual": false,
+		"deepEqual": false,
+		"strictEqual": false,
+		"notStrictEqual": false,
+		"start": false,
+		"stop": false,
+		"expect": false,
+		"raises": false,
+		"ajaxTest": false,
+		"testIframe": false,
+		"testIframeWithCallback": false,
+		"createDashboardXML": false,
+		"createXMLFragment": false,
+		"moduleTeardown": false,
+		"testFoo": false,
+		"url": false,
+		"t": false,
+		"q": false,
+
 		"jQuery": true,
-		"QUnit": true,
 		"module": true,
-		"ok": true,
-		"equal": true,
 		"test": true,
-		"asyncTest": true,
-		"notEqual": true,
-		"deepEqual": true,
-		"strictEqual": true,
-		"notStrictEqual": true,
-		"start": true,
-		"stop": true,
-		"expect": true,
-		"raises": true,
-		"ajaxTest": true,
-		"testIframe": true,
-		"testIframeWithCallback": true,
-		"createDashboardXML": true,
-		"createXMLFragment": true,
-		"moduleTeardown": true,
-		"testFoo": true,
-		"url": true,
-		"t": true,
-		"q": true,
 		"amdDefined": true,
 		"fireNative": true,
 		"Globals": true,