From: Alexander Schmitz <arschmitz@gmail.com>
Date: Mon, 24 Aug 2015 13:36:06 +0000 (-0400)
Subject: Build: Style updates
X-Git-Tag: 1.12.0-beta.1~136
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=3d43f0d7a4ac799b756e500341beb02f439a8d6c;p=jquery-ui.git

Build: Style updates

Ref #14246
---

diff --git a/.jscsrc b/.jscsrc
index de0d7420e..5d6dfd77e 100644
--- a/.jscsrc
+++ b/.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
diff --git a/Gruntfile.js b/Gruntfile.js
index 7797b5c77..0370a9a94 100644
--- a/Gruntfile.js
+++ b/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" ]
diff --git a/tests/unit/effects/core.js b/tests/unit/effects/core.js
index ca7d56034..866c272ee 100644
--- a/tests/unit/effects/core.js
+++ b/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" );
 } );