Use "idiomatic" preset instead of "jquery"

Fixes #76

Author: markelog

.jscsrc

@@ -1,3 +1,3 @@
 {
-    "preset": "jquery"
+    "preset": "idiomatic"
 }

Gruntfile.js

@@ -1,7 +1,7 @@
 module.exports = function( grunt ) {
     "use strict";
 
-    grunt.initConfig( {
+    grunt.initConfig({
         jshint: {
             options: {
                 jshintrc: ".jshintrc"
@@ -15,15 +15,15 @@ module.exports = function( grunt ) {
             methods: "test/methods.js",
             enmasse: "test/enmasse.js"
         }
-    } );
+    });
 
     // Load grunt tasks from NPM packages
     require( "load-grunt-tasks" )( grunt );
     require( "time-grunt" )( grunt );
 
     grunt.loadTasks( "tasks" );
 
-    grunt.registerTask( "lint", [ "jshint", "jscs" ] );
-    grunt.registerTask( "test", [ "lint", "nodeunit" ] );
+    grunt.registerTask( "lint", [ "jshint", "jscs" ]);
+    grunt.registerTask( "test", [ "lint", "nodeunit" ]);
     grunt.registerTask( "default", "test" );
 };

tasks/jscs.js

@@ -9,31 +9,31 @@ module.exports = function( grunt ) {
 
     grunt.registerMultiTask( "jscs", "JavaScript Code Style checker", function() {
         var done = this.async(),
-            options = this.options( {
+            options = this.options({
 
                 // `null` is a default value, but its equivalent to `true`,
                 // With this way it's easy to distinguish specified value
                 config: null
-            } ),
+            }),
 
             jscs = new JSCS( options ),
             runs = this.filesSrc.map( jscs.checker.execute, jscs.checker );
 
-        Vow.allResolved( runs ).spread( function() {
+        Vow.allResolved( runs ).spread(function() {
             var results = [];
 
             // Filter unsuccessful promises
             filter.call( arguments, function( promise ) {
                 return promise.isFulfilled();
 
             // Make array of errors
-            } ).forEach( function( promise ) {
+            }).forEach(function( promise ) {
                 results.push.apply( results, promise.valueOf() );
-            } );
+            });
 
             jscs.setErrors( results ).report().notify();
 
             done( options.force || !jscs.count() );
-        } );
-    } );
+        });
+    });
 };

tasks/lib/jscs.js

@@ -31,13 +31,13 @@ exports.init = function( grunt ) {
      * @param {errorsCollection} errorsCollection
      */
     function defaultReporter( errorsCollection ) {
-        errorsCollection.forEach( function( errors ) {
+        errorsCollection.forEach(function( errors ) {
             if ( !errors.isEmpty() ) {
-                errors.getErrorList().forEach( function( error ) {
+                errors.getErrorList().forEach(function( error ) {
                     grunt.log.writeln( errors.explainError( error, true ) );
-                } );
+                });
             }
-        } );
+        });
     }
 
     /**
@@ -63,9 +63,9 @@ exports.init = function( grunt ) {
     JSCS.prototype.execute = function( path ) {
         var run = this.checker.execute( path );
 
-        run.fail( function( error ) {
+        run.fail(function( error ) {
             grunt.warn( error );
-        } );
+        });
 
         return run;
     };
@@ -193,9 +193,9 @@ exports.init = function( grunt ) {
     JSCS.prototype.setErrors = function( errorsCollection ) {
 
         // Filter excluded files ("excludeFiles" option)
-        this._errors = errorsCollection.filter( function( errors ) {
+        this._errors = errorsCollection.filter(function( errors ) {
             return errors;
-        } );
+        });
 
         return this;
     };
@@ -216,9 +216,9 @@ exports.init = function( grunt ) {
     JSCS.prototype.count = function() {
         var result = 0;
 
-        this._errors.forEach( function( errors ) {
+        this._errors.forEach(function( errors ) {
             result += errors.getErrorCount();
-        } );
+        });
 
         return result;
     };
@@ -239,7 +239,7 @@ exports.init = function( grunt ) {
 
                     return hooker.preempt();
                 }
-            } );
+            });
         }
 
         this._result = this._reporter( this._errors );

test/enmasse.js

@@ -5,73 +5,73 @@ var grunt = require( "grunt" );
 grunt.file.setBase( "test/enmasse" );
 
 exports.fail = function( test ) {
-    grunt.util.spawn( {
+    grunt.util.spawn({
         cmd: "grunt",
         args: [ "jscs:fail" ]
     }, function( error, result ) {
         test.equal( result.code, 3 );
 
         test.done();
-    } );
+    });
 };
 
 exports.broken = function( test ) {
-    grunt.util.spawn( {
+    grunt.util.spawn({
         cmd: "grunt",
         args: [ "jscs:broken" ]
     }, function( error, result ) {
         test.equal( result.code, 3 );
 
         test.done();
-    } );
+    });
 };
 
 exports.force = function( test ) {
-    grunt.util.spawn( {
+    grunt.util.spawn({
         cmd: "grunt",
         args: [ "jscs:force" ]
     }, function( error, result ) {
         test.equal( result.code, 0 );
 
         test.done();
-    } );
+    });
 };
 
 exports.forceAndFatal = function( test ) {
-    grunt.util.spawn( {
+    grunt.util.spawn({
         cmd: "grunt",
         args: [ "jscs:force", "fatal" ]
     }, function( error, result ) {
         test.equal( result.code, 1 );
 
         test.done();
-    } );
+    });
 };
 
 exports.success = function( test ) {
-    grunt.util.spawn( {
+    grunt.util.spawn({
         cmd: "grunt",
         args: [ "jscs:success" ]
     }, function( error, result ) {
         test.equal( result.code, 0 );
 
         test.done();
-    } );
+    });
 };
 
 exports.onlyInline = function( test ) {
-    grunt.util.spawn( {
+    grunt.util.spawn({
         cmd: "grunt",
         args: [ "jscs:only-inline" ]
     }, function( error, result ) {
         test.equal( result.code, 0 );
 
         test.done();
-    } );
+    });
 };
 
 exports.merge = function( test ) {
-    grunt.util.spawn( {
+    grunt.util.spawn({
         cmd: "grunt",
         args: [ "jscs:merge" ]
     }, function( error, result ) {
@@ -80,11 +80,11 @@ exports.merge = function( test ) {
         test.equal( result.code, 3 );
 
         test.done();
-    } );
+    });
 };
 
 exports.mergeWithPath = function( test ) {
-    grunt.util.spawn( {
+    grunt.util.spawn({
         cmd: "grunt",
         args: [ "jscs:merge-with-path" ]
     }, function( error, result ) {
@@ -93,11 +93,11 @@ exports.mergeWithPath = function( test ) {
         test.equal( result.code, 3 );
 
         test.done();
-    } );
+    });
 };
 
 exports.dot = function( test ) {
-    grunt.util.spawn( {
+    grunt.util.spawn({
         cmd: "grunt",
         args: [ "jscs:dot" ]
     }, function( error, result ) {
@@ -107,7 +107,7 @@ exports.dot = function( test ) {
         test.equal( result.stdout.split( "Illegal keyword: with" ).length, 3 );
 
         test.done();
-    } );
+    });
 };
 
 exports.fix = fixable( true );
@@ -116,7 +116,7 @@ exports.fixFail = fixable( false );
 function fixable( expectFailure ) {
     return function( test ) {
         grunt.file.copy( "../fixtures/fixable.source.js", "../fixtures/fixable.js" );
-        grunt.util.spawn( {
+        grunt.util.spawn({
             cmd: "grunt",
             args: [ expectFailure ? "jscs:fix-fail" : "jscs:fix" ]
         }, function( error, result ) {
@@ -130,6 +130,6 @@ function fixable( expectFailure ) {
             }
 
             test.done();
-        } );
+        });
     };
 }

test/enmasse/Gruntfile.js

@@ -1,7 +1,7 @@
 "use strict";
 
 module.exports = function( grunt ) {
-    grunt.initConfig( {
+    grunt.initConfig({
         jscs: {
             fail: {
                 files: {
@@ -102,11 +102,11 @@ module.exports = function( grunt ) {
                 }
             }
         }
-    } );
+    });
 
     grunt.loadTasks( "../../tasks" );
     grunt.registerTask( "default", "jscs" );
     grunt.registerTask( "fatal", function() {
         grunt.fatal( "test" );
-    } );
+    });
 };