4 ms·
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
by robotkilla 12y 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.
- kieranajp 12y agoline 4 col 80 Expected '===' and instead saw '=='. Think this one is due to coffeescript's `if var?` syntax, which compiles to a loose comparison.
- phpnode 12y agoyep, you can disable this warning using .jshintrc - http://www.jshint.com/docs/options/#eqnull http://www.jshint.com/docs/options/#eqnull
- fgribreau 12y agoIndeed, if no .jshintrc was present, check-build will fallback on https://github.com/FGRibreau/check-build/blob/master/defaults/.jshintrc https://github.com/FGRibreau/check-build/blob/master/default...