Build: Style updates

Ref #14246

Author: arschmitz

.jscsrc

@@ -1,8 +1,14 @@
 {
 	"preset": "jquery",
 
-	// disabled until `widget_slice` et al are addressed
-	"requireCamelCaseOrUpperCaseIdentifiers": null,
+	// This is currently unenforcable see https://github.com/jscs-dev/node-jscs/issues/1686
+	"requireCapitalizedComments": null,
+
+	// Until we drop IE8 this prevents things like warning on float keyword
+	"es3": true,
+
+	// We want to output all errors
+	"maxErrors": 1000000,
 
 	// Ref https://github.com/jquery/contribute.jquery.org/issues/80#issuecomment-45253460
 	"maximumLineLength": null

Gruntfile.js

@@ -139,7 +139,10 @@ grunt.initConfig({
 	jscs: {
 		all: {
 			options: {
-				requireCapitalizedComments: null
+				requireCapitalizedComments: null,
+				fix: true,
+				maxErrors: 5000000,
+				config: ".jscsrc"
 			},
 			files: {
 				src: [ "demos/**/*.js", "build/**/*.js", "tests/**/*.js", "ui/**/*.js" ]

tests/unit/effects/core.js

@@ -253,7 +253,7 @@ test( "createPlaceholder: preserves layout affecting properties", function() {
 	deepEqual( before.position.top - position, placeholder.position().top, "position top preserved" );
 	deepEqual( before.position.left - position, placeholder.position().left, "position left preserved" );
 
-	deepEqual( before.float, placeholder.css( "float" ), "float preserved" );
+	deepEqual( before[ "float" ], placeholder.css( "float" ), "float preserved" );
 	deepEqual( before.outerWidth, placeholder.outerWidth( true ), "width preserved" );
 	deepEqual( before.outerHeight, placeholder.outerHeight( true ), "height preserved" );
 } );