Build: Style updates

Ref #14246

diff --git a/.jscsrc b/.jscsrc
index de0d7420e6404bfc4a8019c35dad907181399e3a..5d6dfd77e6e223da45312be4a9c0f0bd6e647068 100644 (file)
--- 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 7797b5c777c89c5d5a95b9ca8aae329bc0b34502..0370a9a94c4625438b2a5b2907d2d0c412fad861 100644 (file)
--- 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 ca7d560340a6c9991acd24ad031f46b1458b187c..866c272eebda3f3e3c55131924fb5469dc4a4105 100644 (file)
--- 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" );
 } );