Build: Upgrade to the latest Grunt and JSHint - Closes jquerygh-786

Author: mikesherov

.jshintrc

@@ -3,10 +3,9 @@
 	"eqnull": true,
 	"eqeqeq": true,
 	"expr": true,
-	"latedef": true,
 	"noarg": true,
+	"node": true,
 	"onevar": true,
-	"smarttabs": true,
 	"trailing": true,
 	"undef": true
 }

build/.jshintrc

@@ -1,6 +1,8 @@
 #!/usr/bin/env node
 /*global cat:true cd:true cp:true echo:true exec:true exit:true ls:true*/
 
+"use strict";
+
 var baseDir, repoDir, prevVersion, newVersion, nextVersion, tagTime,
 	fs = require( "fs" ),
 	path = require( "path" ),

build/release/release.js

@@ -1,6 +1,9 @@
 module.exports = function( grunt ) {
 
-var path = require( "path" );
+"use strict";
+
+var path = require( "path" ),
+	fs = require( "fs" );
 
 grunt.registerTask( "manifest", "Generate jquery.json manifest files", function() {
 	var pkg = grunt.config( "pkg" ),

build/tasks/build.js

@@ -1,6 +1,7 @@
-/*jshint node: true */
 module.exports = function( grunt ) {
 
+"use strict";
+
 var versions = {
 		"git": "git",
 		"1.8": "1.8.0 1.8.1 1.8.2",

build/tasks/testswarm.js

@@ -1,6 +1,7 @@
-/*jshint node: true */
 module.exports = function( grunt ) {
 
+"use strict";
+
 var
 	// files
 	coreFiles = [
@@ -326,9 +327,7 @@ grunt.initConfig({
 		}
 
 		return {
-			// TODO: use "faux strict mode" https://github.com/jshint/jshint/issues/504
-			// TODO: limit `smarttabs` to multi-line comments https://github.com/jshint/jshint/issues/503
-			options: parserc(),
+			grunt: parserc(),
 			ui: parserc( "ui/" ),
 			// TODO: `evil: true` is only for document.write() https://github.com/jshint/jshint/issues/519
 			// TODO: don't create so many globals in tests

grunt.js

@@ -48,7 +48,7 @@
 	],
 	"dependencies": {},
 	"devDependencies": {
-		"grunt": "~0.3.9",
+		"grunt": "~0.3.17",
 		"grunt-css": "0.2.0",
 		"grunt-compare-size": "0.1.4",
 		"grunt-html": "0.1.1",

package.json

@@ -9,7 +9,6 @@
 	"latedef": true,
 	"noarg": true,
 	"onevar": true,
-	"smarttabs": true,
 	"trailing": true,
 	"undef": true,
 	"predef": [

tests/.jshintrc

@@ -7,9 +7,9 @@ module("datepicker: tickets");
 
 // http://forum.jquery.com/topic/several-breaking-changes-in-jquery-ui-1-8rc1
 test('beforeShowDay-getDate', function() {
-    expect( 3 );
+	expect( 3 );
 	var inp = init('#inp', {beforeShowDay: function(date) { inp.datepicker('getDate'); return [true, '']; }}),
-	   dp = $('#ui-datepicker-div');
+		dp = $('#ui-datepicker-div');
 	inp.val('01/01/2010').datepicker('show');
 	// contains non-breaking space
 	equal($('div.ui-datepicker-title').text(), 'January 2010', 'Initial month');
@@ -26,63 +26,63 @@ test('beforeShowDay-getDate', function() {
 });
 
 test('Ticket 7602: Stop datepicker from appearing with beforeShow event handler', function(){
-    expect( 3 );
-    var inp = init('#inp',{
-            beforeShow: function(){
-                return false;
-            }
-        }),
-        dp = $('#ui-datepicker-div');
-    inp.datepicker('show');
-    equal(dp.css('display'), 'none',"beforeShow returns false");
-    inp.datepicker('destroy');
+	expect( 3 );
+	var inp = init('#inp',{
+			beforeShow: function(){
+				return false;
+			}
+		}),
+		dp = $('#ui-datepicker-div');
+	inp.datepicker('show');
+	equal(dp.css('display'), 'none',"beforeShow returns false");
+	inp.datepicker('destroy');
 
-    inp = init('#inp',{
-        beforeShow: function(){
-        }
-    });
-    dp = $('#ui-datepicker-div');
-    inp.datepicker('show');
-    equal(dp.css('display'), 'block',"beforeShow returns nothing");
+	inp = init('#inp',{
+		beforeShow: function(){
+		}
+	});
+	dp = $('#ui-datepicker-div');
+	inp.datepicker('show');
+	equal(dp.css('display'), 'block',"beforeShow returns nothing");
 	inp.datepicker('hide');
-    inp.datepicker('destroy');
+	inp.datepicker('destroy');
 
-    inp = init('#inp',{
-        beforeShow: function(){
-            return true;
-        }
-    });
-    dp = $('#ui-datepicker-div');
-    inp.datepicker('show');
-    equal(dp.css('display'), 'block',"beforeShow returns true");
+	inp = init('#inp',{
+		beforeShow: function(){
+			return true;
+		}
+	});
+	dp = $('#ui-datepicker-div');
+	inp.datepicker('show');
+	equal(dp.css('display'), 'block',"beforeShow returns true");
 	inp.datepicker('hide');
-    inp.datepicker('destroy');
+	inp.datepicker('destroy');
 });
 
 test('Ticket 6827: formatDate day of year calculation is wrong during day lights savings time', function(){
-    expect( 1 );
-    var time = $.datepicker.formatDate("oo", new Date("2010/03/30 12:00:00 CDT"));
-    equal(time, "089");
+	expect( 1 );
+	var time = $.datepicker.formatDate("oo", new Date("2010/03/30 12:00:00 CDT"));
+	equal(time, "089");
 });
 
 test('Ticket #7244: date parser does not fail when too many numbers are passed into the date function', function() {
-    expect( 4 );
-    var date;
-    try{
-        date = $.datepicker.parseDate('dd/mm/yy', '18/04/19881');
-        ok(false, "Did not properly detect an invalid date");
-    }catch(e){
-        ok("invalid date detected");
-    }
+	expect( 4 );
+	var date;
+	try{
+		date = $.datepicker.parseDate('dd/mm/yy', '18/04/19881');
+		ok(false, "Did not properly detect an invalid date");
+	}catch(e){
+		ok("invalid date detected");
+	}
 
-    try {
-      date = $.datepicker.parseDate('dd/mm/yy', '18/04/1988 @ 2:43 pm');
-      equal(date.getDate(), 18);
-      equal(date.getMonth(), 3);
-      equal(date.getFullYear(), 1988);
-    } catch(e) {
-      ok(false, "Did not properly parse date with extra text separated by whitespace");
-    }
+	try {
+		date = $.datepicker.parseDate('dd/mm/yy', '18/04/1988 @ 2:43 pm');
+		equal(date.getDate(), 18);
+		equal(date.getMonth(), 3);
+		equal(date.getFullYear(), 1988);
+	} catch(e) {
+		ok(false, "Did not properly parse date with extra text separated by whitespace");
+	}
 });
 
 })(jQuery);

tests/unit/datepicker/datepicker_tickets.js

@@ -12,6 +12,10 @@ function includeScript( url ) {
 	document.write( "<script src='../../../" + url + "'></script>" );
 }
 
+function url( value ) {
+	return value + (/\?/.test(value) ? "&" : "?") + new Date().getTime() + "" + parseInt(Math.random() * 100000, 10);
+}
+
 reset = QUnit.reset;
 QUnit.reset = function() {
 	// Ensure jQuery events and data on the fixture are properly removed
@@ -65,11 +69,11 @@ TestHelpers.testJshint = function( module ) {
 
 		$.when(
 			$.ajax({
-				url: "../../../ui/.jshintrc",
+				url: url("../../../ui/.jshintrc"),
 				dataType: "json"
 			}),
 			$.ajax({
-				url: "../../../ui/jquery.ui." + module + ".js",
+				url: url("../../../ui/jquery.ui." + module + ".js"),
 				dataType: "text"
 			})
 		).done(function( hintArgs, srcArgs ) {