vladikoff / test-grunt-jscs Goto Github PK

➜  test-grunt-jscs git:(master) grunt jscs
Running "jscs:src" (jscs) task
Multiple var declaration at node_modules/grunt/lib/grunt.js :
    68 |
    69 |  // Display the grunt version and quit if the user did --version.
    70 |  var _tasks, _options;
----------^
    71 |  if (option('version')) {
    72 |    // Not --verbose.
Line must be at most 120 characters at node_modules/grunt/lib/grunt/cli.js :
    47 |  },
    48 |  base: {
    49 |    info: 'Specify an alternate base path. By default, all file paths are relative to the Gruntfile. (grunt.file.setBase) *',
-------------------------------------------------------------------------------------------------------------------------------------^
    50 |    type: path
    51 |  },
Line must be at most 120 characters at node_modules/grunt/lib/grunt/cli.js :
    56 |  },
    57 |  gruntfile: {
    58 |    info: 'Specify an alternate Gruntfile. By default, grunt looks in the current or parent directories for the nearest Gruntfile.js or Gruntfile.coffee file.',
------------------------------------------------------------------------------------------------------------------------------------------------------------------------^
    59 |    type: path
    60 |  },
Should be one space instead of 0, after "catch" keyword at node_modules/grunt/lib/grunt/file.js :
   123 |          return fs.statSync(filepath)[options.filter]();
   124 |        }
   125 |      } catch(e) {
---------------------^
   126 |        // Otherwise, it's probably not the right type.
   127 |        return false;
Should be one space instead of 0, after "catch" keyword at node_modules/grunt/lib/grunt/file.js :
   199 |      try {
   200 |        fs.mkdirSync(subpath, mode);
   201 |      } catch(e) {
---------------------^
   202 |        throw grunt.util.error('Unable to create directory "' + subpath + '" (Error code: ' + e.code + ').', e);
   203 |      }
Should be one space instead of 0, after "catch" keyword at node_modules/grunt/lib/grunt/file.js :
   243 |    grunt.verbose.ok();
   244 |    return contents;
   245 |  } catch(e) {
-----------------^
   246 |    grunt.verbose.error();
   247 |    throw grunt.util.error('Unable to read "' + filepath + '" file (Error code: ' + e.code + ').', e);
Should be one space instead of 0, after "catch" keyword at node_modules/grunt/lib/grunt/file.js :
   258 |    grunt.verbose.ok();
   259 |    return result;
   260 |  } catch(e) {
-----------------^
   261 |    grunt.verbose.error();
   262 |    throw grunt.util.error('Unable to parse "' + filepath + '" file (' + e.message + ').', e);
Should be one space instead of 0, after "catch" keyword at node_modules/grunt/lib/grunt/file.js :
   273 |    grunt.verbose.ok();
   274 |    return result;
   275 |  } catch(e) {
-----------------^
   276 |    grunt.verbose.error();
   277 |    throw grunt.util.error('Unable to parse "' + filepath + '" file (' + e.problem + ').', e);
Should be one space instead of 0, after "catch" keyword at node_modules/grunt/lib/grunt/file.js :
   298 |    grunt.verbose.ok();
   299 |    return true;
   300 |  } catch(e) {
-----------------^
   301 |    grunt.verbose.error();
   302 |    throw grunt.util.error('Unable to write "' + filepath + '" file (Error code: ' + e.code + ').', e);
Should be one space instead of 0, after "catch" keyword at node_modules/grunt/lib/grunt/file.js :
   321 |      contents = options.process(contents, srcpath);
   322 |      grunt.verbose.ok();
   323 |    } catch(e) {
-------------------^
   324 |      grunt.verbose.error();
   325 |      throw grunt.util.error('Error while processing "' + srcpath + '" file.', e);
Should be one space instead of 0, after "catch" keyword at node_modules/grunt/lib/grunt/file.js :
   371 |    grunt.verbose.ok();
   372 |    return true;
   373 |  } catch(e) {
-----------------^
   374 |    grunt.verbose.error();
   375 |    throw grunt.util.error('Unable to delete "' + filepath + '" file (' + e.message + ').', e);
Should be one space instead of 0, after "catch" keyword at node_modules/grunt/lib/grunt/file.js :
   433 |  try {
   434 |    return file.arePathsEquivalent(fs.realpathSync(process.cwd()), fs.realpathSync(filepath));
   435 |  } catch(e) {
-----------------^
   436 |    return false;
   437 |  }
Should be one space instead of 0, after "catch" keyword at node_modules/grunt/lib/grunt/file.js :
   443 |  try {
   444 |    return file.doesPathContain(fs.realpathSync(process.cwd()), fs.realpathSync(filepath));
   445 |  } catch(e) {
-----------------^
   446 |    return false;
   447 |  }
Multiple line break at node_modules/grunt/lib/grunt/help.js :
    49 |  exports.queue.forEach(function(name) { exports[name](); });
    50 |};
    51 |
--------^
    52 |
    53 |// Header.
Multiple var declaration at node_modules/grunt/lib/grunt/task.js :
    92 |// Normalize multi task files.
    93 |task.normalizeMultiTaskFiles = function(data, target) {
    94 |  var prop, obj;
----------^
    95 |  var files = [];
    96 |  if (grunt.util.kindOf(data) === 'object') {
Should be one space instead of 0, after "catch" keyword at node_modules/grunt/lib/grunt/task.js :
   287 |    // Return result.
   288 |    return result;
   289 |  } catch(e) {
-----------------^
   290 |    grunt.log.error(e.message);
   291 |  }
Should be one space instead of 0, after "catch" keyword at node_modules/grunt/lib/grunt/task.js :
   337 |      grunt.verbose.warn('No tasks were registered or unregistered.');
   338 |    }
   339 |  } catch(e) {
-----------------^
   340 |    // Something went wrong.
   341 |    grunt.log.write(msg).error().verbose.error(e.stack).or.error(e);
Should be one space instead of 0, after "catch" keyword at node_modules/grunt/lib/grunt/task.js :
   361 |      loadTask(path.join(tasksdir, filename));
   362 |    });
   363 |  } catch(e) {
-----------------^
   364 |    grunt.log.verbose.error(e.stack).or.error(e);
   365 |  }
Illegal space before opening round brace at node_modules/grunt/lib/util/task.js :
   224 |      // see: https://github.com/gruntjs/grunt/pull/1026
   225 |      if (asyncDone) {
   226 |        process.nextTick(function () {
---------------------------------^
   227 |          done(err, success);
   228 |        });
All identifiers must be camelCase or UPPER_CASE at node_modules/grunt-contrib-concat/tasks/lib/sourcemap.js :
   171 |  // Return a string for inline use or write the source map to disk.
   172 |  SourceMapConcatHelper.prototype._write = function() {
   173 |    var code_map = this.node.toStringWithSourceMap({
----------------^
   174 |      file: path.relative(path.dirname(this.dest), this.files.dest)
   175 |    });
All identifiers must be camelCase or UPPER_CASE at node_modules/grunt-contrib-concat/tasks/lib/sourcemap.js :
   176 |    // Consume the new sourcemap.
   177 |    var generator = SourceMapGenerator.fromSourceMap(
   178 |      new SourceMapConsumer(code_map.map.toJSON())
------------------------------------^
   179 |    );
   180 |    // Consume sourcemaps for source files.
Missing space before opening curly brace for block expressions at node_modules/grunt-contrib-connect/tasks/connect.js :
   112 |    async.waterfall([
   113 |      // find a port for livereload if needed first
   114 |      function(callback){
-------------------------------^
   115 |
   116 |        // Inject live reload snippet
Missing space before opening curly brace at node_modules/grunt-contrib-connect/tasks/connect.js :
   112 |    async.waterfall([
   113 |      // find a port for livereload if needed first
   114 |      function(callback){
-------------------------------^
   115 |
   116 |        // Inject live reload snippet
Missing space before opening curly brace for block expressions at node_modules/grunt-contrib-connect/tasks/connect.js :
   127 |        }
   128 |      },
   129 |      function(){
-----------------------^
   130 |
   131 |        var app = connect.apply(null, middleware);
Missing space before opening curly brace at node_modules/grunt-contrib-connect/tasks/connect.js :
   127 |        }
   128 |      },
   129 |      function(){
-----------------------^
   130 |
   131 |        var app = connect.apply(null, middleware);
Line must be at most 120 characters at node_modules/grunt-contrib-connect/tasks/connect.js :
   150 |        }
   151 |
   152 |        portscanner.findAPortNotInUse(options.port, options.port + MAX_PORTS, options.hostname, function(error, foundPort) {
------------------------------------------------------------------------------------------------------------------------------------^
   153 |          // if the found port doesn't match the option port, and we are forced to use the option port
   154 |          if (options.port !== foundPort && options.useAvailablePort === false) {
Operator , should not stick to following expression at node_modules/grunt-contrib-jshint/tasks/jshint.js :
    49 |        if (jshint.usingGruntReporter === true) {
    50 |
    51 |          var numErrors = grunt.util._.reduce(results,function(memo,result){
-------------------------------------------------------------^
    52 |            return memo + (result.error ? 1 : 0);
    53 |          },0);
Operator , should not stick to following expression at node_modules/grunt-contrib-jshint/tasks/jshint.js :
    49 |        if (jshint.usingGruntReporter === true) {
    50 |
    51 |          var numErrors = grunt.util._.reduce(results,function(memo,result){
---------------------------------------------------------------------------^
    52 |            return memo + (result.error ? 1 : 0);
    53 |          },0);
Missing space before opening curly brace for block expressions at node_modules/grunt-contrib-jshint/tasks/jshint.js :
    49 |        if (jshint.usingGruntReporter === true) {
    50 |
    51 |          var numErrors = grunt.util._.reduce(results,function(memo,result){
----------------------------------------------------------------------------------^
    52 |            return memo + (result.error ? 1 : 0);
    53 |          },0);
Missing space before opening curly brace at node_modules/grunt-contrib-jshint/tasks/jshint.js :
    49 |        if (jshint.usingGruntReporter === true) {
    50 |
    51 |          var numErrors = grunt.util._.reduce(results,function(memo,result){
----------------------------------------------------------------------------------^
    52 |            return memo + (result.error ? 1 : 0);
    53 |          },0);
Operator , should not stick to following expression at node_modules/grunt-contrib-jshint/tasks/jshint.js :
    51 |          var numErrors = grunt.util._.reduce(results,function(memo,result){
    52 |            return memo + (result.error ? 1 : 0);
    53 |          },0);
-------------------^
    54 |
    55 |          var numFiles = data.length;
Operator , should not stick to following expression at node_modules/grunt-contrib-jshint/tasks/jshint.js :
    54 |
    55 |          var numFiles = data.length;
    56 |          grunt.log.error(numErrors + ' ' + grunt.util.pluralize(numErrors,'error/errors') + ' in ' +
----------------------------------------------------------------------------------^
    57 |                          numFiles + ' ' + grunt.util.pluralize(numFiles,'file/files'));
    58 |        }
Operator , should not stick to following expression at node_modules/grunt-contrib-jshint/tasks/jshint.js :
    55 |          var numFiles = data.length;
    56 |          grunt.log.error(numErrors + ' ' + grunt.util.pluralize(numErrors,'error/errors') + ' in ' +
    57 |                          numFiles + ' ' + grunt.util.pluralize(numFiles,'file/files'));
--------------------------------------------------------------------------------^
    58 |        }
    59 |      } else {
Operator , should not stick to following expression at node_modules/grunt-contrib-jshint/tasks/jshint.js :
    59 |      } else {
    60 |        if (jshint.usingGruntReporter === true && data.length > 0) {
    61 |          grunt.log.ok(data.length + ' ' + grunt.util.pluralize(data.length,'file/files') + ' lint free.');
-----------------------------------------------------------------------------------^
    62 |        }
    63 |      }
Operator , should not stick to following expression at node_modules/grunt-contrib-jshint/tasks/lib/jshint.js :
    18 |  };
    19 |
    20 |  var pad = function(msg,length) {
--------------------------------^
    21 |    while (msg.length < length) {
    22 |      msg = ' ' + msg;
Operator , should not stick to following expression at node_modules/grunt-contrib-jshint/tasks/lib/jshint.js :
   111 |        // Replacing tabs with appropriate spaces (i.e. columns) ensures that
   112 |        // caret will line up correctly.
   113 |        var evidence = e.evidence.replace(/\t/g,grunt.util.repeat(options.indent,' '));
-------------------------------------------------------^
   114 |
   115 |        grunt.log.writeln((pad(e.line.toString(),7) + ' |') + evidence.grey);
Operator , should not stick to following expression at node_modules/grunt-contrib-jshint/tasks/lib/jshint.js :
   111 |        // Replacing tabs with appropriate spaces (i.e. columns) ensures that
   112 |        // caret will line up correctly.
   113 |        var evidence = e.evidence.replace(/\t/g,grunt.util.repeat(options.indent,' '));
----------------------------------------------------------------------------------------^
   114 |
   115 |        grunt.log.writeln((pad(e.line.toString(),7) + ' |') + evidence.grey);
Operator , should not stick to following expression at node_modules/grunt-contrib-jshint/tasks/lib/jshint.js :
   113 |        var evidence = e.evidence.replace(/\t/g,grunt.util.repeat(options.indent,' '));
   114 |
   115 |        grunt.log.writeln((pad(e.line.toString(),7) + ' |') + evidence.grey);
--------------------------------------------------------^
   116 |        grunt.log.write(grunt.util.repeat(9,' ') + grunt.util.repeat(e.character -1,' ') + '^ ');
   117 |        grunt.verbose.write('[' + e.code + '] ');
Operator , should not stick to following expression at node_modules/grunt-contrib-jshint/tasks/lib/jshint.js :
   114 |
   115 |        grunt.log.writeln((pad(e.line.toString(),7) + ' |') + evidence.grey);
   116 |        grunt.log.write(grunt.util.repeat(9,' ') + grunt.util.repeat(e.character -1,' ') + '^ ');
---------------------------------------------------^
   117 |        grunt.verbose.write('[' + e.code + '] ');
   118 |        grunt.log.writeln(e.reason);
Operator - should not stick to following expression at node_modules/grunt-contrib-jshint/tasks/lib/jshint.js :
   114 |
   115 |        grunt.log.writeln((pad(e.line.toString(),7) + ' |') + evidence.grey);
   116 |        grunt.log.write(grunt.util.repeat(9,' ') + grunt.util.repeat(e.character -1,' ') + '^ ');
-----------------------------------------------------------------------------------------^
   117 |        grunt.verbose.write('[' + e.code + '] ');
   118 |        grunt.log.writeln(e.reason);
Operator , should not stick to following expression at node_modules/grunt-contrib-jshint/tasks/lib/jshint.js :
   114 |
   115 |        grunt.log.writeln((pad(e.line.toString(),7) + ' |') + evidence.grey);
   116 |        grunt.log.write(grunt.util.repeat(9,' ') + grunt.util.repeat(e.character -1,' ') + '^ ');
-------------------------------------------------------------------------------------------^
   117 |        grunt.verbose.write('[' + e.code + '] ');
   118 |        grunt.log.writeln(e.reason);
>> 41 code style errors found!
Warning: Task "jscs:src" failed. Use --force to continue.

Aborted due to warnings.
➜  test-grunt-jscs git:(master)