cordova-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From agri...@apache.org
Subject [3/3] git commit: Enable jshint for node modules as well
Date Fri, 23 May 2014 18:06:04 GMT
Enable jshint for node modules as well


Project: http://git-wip-us.apache.org/repos/asf/cordova-app-harness/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-app-harness/commit/e18f4bfe
Tree: http://git-wip-us.apache.org/repos/asf/cordova-app-harness/tree/e18f4bfe
Diff: http://git-wip-us.apache.org/repos/asf/cordova-app-harness/diff/e18f4bfe

Branch: refs/heads/master
Commit: e18f4bfed84b701b54bd43f084c864598d0a26fa
Parents: e82ef5c
Author: Andrew Grieve <agrieve@chromium.org>
Authored: Fri May 23 14:05:44 2014 -0400
Committer: Andrew Grieve <agrieve@chromium.org>
Committed: Fri May 23 14:05:44 2014 -0400

----------------------------------------------------------------------
 .gitignore | 4 +++-
 .jshintrc  | 3 ++-
 checkjs.sh | 5 ++++-
 3 files changed, 9 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cordova-app-harness/blob/e18f4bfe/.gitignore
----------------------------------------------------------------------
diff --git a/.gitignore b/.gitignore
index ea23fd6..8516c5e 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,2 +1,4 @@
 .DS_Store
-harness-push/node_modules
+harness-push/node_modules/*
+!harness-push/node_modules/cordova-harness-client
+harness-push/node_modules/cordova-harness-client/node_modules

http://git-wip-us.apache.org/repos/asf/cordova-app-harness/blob/e18f4bfe/.jshintrc
----------------------------------------------------------------------
diff --git a/.jshintrc b/.jshintrc
index 227f5fc..91a9a7d 100644
--- a/.jshintrc
+++ b/.jshintrc
@@ -4,7 +4,7 @@
     "camelcase": true,
     "curly": true,
     "eqeqeq": false,
-    "forin": true,
+    "forin": false,
     "immed": true,
     "indent": 4,
     "latedef": false,
@@ -24,6 +24,7 @@
     "sub":true,
 
     //environment
+    "node": true,
     "browser": true,
     "devel":true,
 

http://git-wip-us.apache.org/repos/asf/cordova-app-harness/blob/e18f4bfe/checkjs.sh
----------------------------------------------------------------------
diff --git a/checkjs.sh b/checkjs.sh
index 8c1b59b..afb7173 100755
--- a/checkjs.sh
+++ b/checkjs.sh
@@ -28,4 +28,7 @@ if ! which jshint; then
 fi
 echo "Running jsHint"
 cd $(dirname "$0")
-jshint www/ --exclude www/cdvah/js/libs --verbose --show-non-errors
+set -x
+jshint www/ --exclude www/cdvah/js/libs --verbose --show-non-errors && \
+jshint harness-push/ --exclude harness-push/node_modules --verbose --show-non-errors &&
\
+jshint harness-push/node_modules/cordova-harness-client --exclude harness-push/node_modules/cordova-harness-client/node_modules
--verbose --show-non-errors


Mime
View raw message