test-grunt-jscs's Issues
Test run
➜ 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)
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.