Merge pull request #184 from kberg/master

[dygraphs.git] / lint.sh

diff --git a/lint.sh b/lint.sh
index 6c640ad..a45d901 100755 (executable)
--- 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)...'