diff options
| -rw-r--r-- | .jscsrc | 10 | ||||
| -rw-r--r-- | Gruntfile.js | 5 | ||||
| -rw-r--r-- | tests/unit/effects/core.js | 2 |
3 files changed, 13 insertions, 4 deletions
@@ -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" ); } ); |