Hacker News new | ask | show | jobs
by robotkilla 4255 days ago
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.

1 comments

    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.
yep, you can disable this warning using .jshintrc - http://www.jshint.com/docs/options/#eqnull
Indeed, if no .jshintrc was present, check-build will fallback on https://github.com/FGRibreau/check-build/blob/master/default...