News for hackers. Stuff that matters.
Re:Check that your Node.js project is well written, secure a... by robotkilla on Monday October 20, 2014 @07:36AM (#8481580)
Interesting library – I ran it on one of my projects which consists of JS compiled from literate coffeescript. The JS also has also been run through browserify and uglify.

My code is tested using mocha (albeit lightly), passes lint tests and seems to have no issues pre or post compiling, however when I run check-build I receive 28 errors and 25 warnings.

Here is a sample of check-build's output:

    line 4   col 448    Line is too long.
    line 4   col 22     Missing "use strict" statement.
    line 4   col 80     Expected '===' and instead saw '=='.
    line 4   col 99     Missing "use strict" statement.
    line 4   col 109    Expected '{' and instead saw 'return'.
    line 4   col 123    Missing "use strict" statement.
    line 4   col 124    Missing "use strict" statement.
This seems to be directly related to using uglify – my guess is that I need to tweak the config file to fit my stack (I'm using the default example).

edit - I already changed the config to match my directory structure and am running check-build on the compiled JS.

Re:Check that your Node.js project is well written, secure a... by kieranajp on Monday October 20, 2014 @08:41AM (#8481783)

    line 4   col 80     Expected '===' and instead saw '=='.
Think this one is due to coffeescript's `if var?` syntax, which compiles to a loose comparison.
Re:Check that your Node.js project is well written, secure a... by phpnode on Monday October 20, 2014 @09:21AM (#8481962)
yep, you can disable this warning using .jshintrc - http://www.jshint.com/docs/options/#eqnull
Re:Check that your Node.js project is well written, secure a... by fgribreau on Monday October 20, 2014 @11:23AM (#8482556)
Indeed, if no .jshintrc was present, check-build will fallback on https://github.com/FGRibreau/check-build/blob/master/default...