Add jshintrc and inline lint checking to node-red-nodes

This commit is contained in:
dceejay 2015-04-16 10:56:22 +01:00
parent cd57346cae
commit 1e7717ac8a
3 changed files with 18 additions and 2 deletions

13
.jshintrc Normal file
View File

@ -0,0 +1,13 @@
{
"asi": true, // allow missing semicolons
"curly": true, // require braces
"eqnull": true, // ignore ==null
"forin": true, // require property filtering in "for in" loops
"immed": true, // require immediate functions to be wrapped in ( )
"nonbsp": true, // warn on unexpected whitespace breaking chars
//"strict": true, // commented out for now as it causes 100s of warnings, but want to get there eventually
"unused": true, // Check for unused functions and variables
"loopfunc": true, // allow functions to be defined in loops
//"expr": true, // allow ternery operator syntax...
"sub": true // don't warn that foo['bar'] should be written as foo.bar
}

View File

@ -39,6 +39,7 @@ module.exports = function(grunt) {
//"strict": true, // commented out for now as it causes 100s of warnings, but want to get there eventually //"strict": true, // commented out for now as it causes 100s of warnings, but want to get there eventually
"loopfunc": true, // allow functions to be defined in loops "loopfunc": true, // allow functions to be defined in loops
"sub": true, // don't warn that foo['bar'] should be written as foo.bar "sub": true, // don't warn that foo['bar'] should be written as foo.bar
//"unused": true, // Check for unused functions
//"forin":false, // turn off check for "for (x in y...)" //"forin":false, // turn off check for "for (x in y...)"
"reporter": require('jshint-stylish') "reporter": require('jshint-stylish')
}, },
@ -55,12 +56,13 @@ module.exports = function(grunt) {
}, },
}, },
inlinelint: { inlinelint: {
html: ['*/*.html'] html: ['*/*/*.html']
} }
}); });
grunt.loadNpmTasks('grunt-simple-mocha'); grunt.loadNpmTasks('grunt-simple-mocha');
grunt.loadNpmTasks('grunt-contrib-jshint'); grunt.loadNpmTasks('grunt-contrib-jshint');
grunt.loadNpmTasks('grunt-lint-inline');
grunt.registerTask('default', ['jshint:all', 'simplemocha:all'] ); grunt.registerTask('default', ['jshint:all', 'inlinelint', 'simplemocha:all'] );
}; };

View File

@ -20,6 +20,7 @@
"grunt-cli": "0.1.13", "grunt-cli": "0.1.13",
"grunt-simple-mocha": "0.4.0", "grunt-simple-mocha": "0.4.0",
"grunt-contrib-jshint": "0.11.0", "grunt-contrib-jshint": "0.11.0",
"grunt-lint-inline": "0.4.3",
"jshint-stylish": "1.0.1", "jshint-stylish": "1.0.1",
"mocha": "2.1.0", "mocha": "2.1.0",
"should": "4.6.5", "should": "4.6.5",