X-Git-Url: https://adrianiainlam.tk/git/?a=blobdiff_plain;f=lint.sh;h=a45d901b5f262226ccffb148f2bb81d744ed0b0f;hb=cb136039d06ace657e202c884f84569322ffdaf2;hp=6c640ad94de19e3714f0a8fdf3be6a74a72d80b8;hpb=fefe0bfe5e8e551e1fdb886408a335139a5f8ba3;p=dygraphs.git diff --git a/lint.sh b/lint.sh index 6c640ad..a45d901 100755 --- a/lint.sh +++ b/lint.sh @@ -5,8 +5,13 @@ # # The zero-argument form lints everything. -jsc_opts='maxerr:10000,devel:true,browser:true' -rhino_opts='maxerr=10000,devel=true,browser=true' +# See jshint/build/jshint-rhino.js for documentation on these parameters. +# devel defines logging globals (i.e. "console.log") +# browser defines standard web browser globals (i.e. "document") +# shadow disables warnings on multiple var definitions in one scope (i.e. two +# loops with "var i") +jsc_opts='maxerr:10000,devel:true,browser:true,shadow:true' +rhino_opts='maxerr=10000,devel=true,browser=true,shadow=true' if [ $# -gt 1 ]; then echo "Usage: $0 [file.js]" @@ -14,11 +19,13 @@ if [ $# -gt 1 ]; then fi if [ $# -eq 0 ]; then - files=$(ls dygraph*.js plugins/*.js | grep -v combined | grep -v dev.js) + files=$(ls dygraph*.js plugins/*.js | grep -v combined | grep -v dev.js| grep -v externs) else files=$1 fi +jshint_opts="shadow=false" + if [ -e /System/Library/Frameworks/JavaScriptCore.framework/Versions/A/Resources/jsc ]; then # use JSC (Safari/JavaScriptCore) to run JSHint -- much faster than Rhino. echo 'Running JSHint w/ JavaScriptCore (jsc)...'