feat: Merge pull request #826 from pelias/repo-crawler-add-dep-check

Add dependency checks to pre-commit and travis

.travis.yml

@@ -1,20 +1,16 @@
 sudo: false
 language: node_js
-cache:
-  directories:
-    - node_modules
 notifications:
   email: false
 node_js:
   - 4
   - 6
 matrix:
-    fast_finish: true
-    allow_failures:
+  fast_finish: true
 env:
   global:
     - CXX=g++-4.8
-script: "npm run travis"
+script: npm run travis
 addons:
   apt:
     sources:
@@ -22,7 +18,7 @@ addons:
     packages:
       - g++-4.8
 before_install:
-  - npm i -g npm@^2.0.0
+  - npm i -g npm@^3.0.0
 before_script:
   - npm prune
 after_success:

package.json

@@ -14,11 +14,12 @@
     "lint": "jshint .",
     "start": "node index.js",
     "test": "npm run unit",
-    "travis": "npm test",
+    "travis": "npm run check-dependencies && npm test",
     "unit": "./bin/units",
     "validate": "npm ls",
     "semantic-release": "semantic-release pre && npm publish && semantic-release post",
-    "config": "node -e \"console.log(JSON.stringify(require( 'pelias-config' ).generate(require('./schema')), null, 2))\""
+    "config": "node -e \"console.log(JSON.stringify(require( 'pelias-config' ).generate(require('./schema')), null, 2))\"",
+    "check-dependencies": "node_modules/.bin/npm-check --production --ignore pelias-interpolation"
   },
   "repository": {
     "type": "git",
@@ -74,6 +75,7 @@
     "difflet": "^1.0.1",
     "istanbul": "^0.4.2",
     "jshint": "^2.5.6",
+    "npm-check": "^5.4.0",
     "nsp": "^2.2.0",
     "precommit-hook": "^3.0.0",
     "proxyquire": "^1.7.10",
@@ -87,6 +89,7 @@
   "pre-commit": [
     "lint",
     "validate",
-    "test"
+    "test",
+    "check-dependencies"
   ]
 }