From 46121edd6202abd4f837c10fa319d68e8f5451d9 Mon Sep 17 00:00:00 2001 From: Nicholas Dudfield Date: Tue, 23 Jun 2015 16:54:57 +0700 Subject: [PATCH] Remove global install for CI dependencies: * Add them to package.json * Fixes weird error with missing `esprima-fb` --- bin/ci.sh | 4 +--- package.json | 3 ++- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/bin/ci.sh b/bin/ci.sh index 7a7c9aae..7eadb7ee 100755 --- a/bin/ci.sh +++ b/bin/ci.sh @@ -10,11 +10,9 @@ typecheck() { lint() { REPO_URL="https://raw.githubusercontent.com/ripple/javascript-style-guide" - npm install -g eslint - npm install -g esprima-fb # allow flow type annotations in eslint curl "$REPO_URL/es6/eslintrc" > ./eslintrc echo "parser: esprima-fb" >> ./eslintrc - eslint --reset -c ./eslintrc $(git --no-pager diff --name-only -M100% --diff-filter=AM --relative $(git merge-base FETCH_HEAD origin/HEAD) FETCH_HEAD | grep "\.js$") + node_modules/.bin/eslint --reset -c ./eslintrc $(git --no-pager diff --name-only -M100% --diff-filter=AM --relative $(git merge-base FETCH_HEAD origin/HEAD) FETCH_HEAD | grep "\.js$") } unittest() { diff --git a/package.json b/package.json index bd16d966..e60d2875 100644 --- a/package.json +++ b/package.json @@ -34,7 +34,8 @@ "babel-core": "^5.5.4", "babel-loader": "^5.0.0", "coveralls": "~2.10.0", - "eslint": "^0.18.0", + "eslint": "^0.23.0", + "esprima-fb": "^15001.1.0-dev-harmony-fb", "eventemitter2": "^0.4.14", "gulp": "~3.8.10", "gulp-bump": "~0.1.13",