Merge pull request #40 from nodenv/lint

linting

Author: jasonkarns

.travis.yml

@@ -1,6 +1,3 @@
 language: node_js
 node_js: node
-# Lint and test in parallel
-script:
-  - npm run lint
-  - npm test
+cache: npm

package.json

@@ -22,9 +22,10 @@
     "test": "test"
   },
   "scripts": {
-    "lint": "git ls-files bin etc libexec | xargs shellcheck",
+    "lint": "git ls-files bin etc libexec test/*.bash | xargs shellcheck",
     "pretest": "[ -d $npm_package_directories_testnodenv ] || git clone --quiet --depth 1 https://github.com/nodenv/nodenv $npm_package_directories_testnodenv",
     "test": "PATH=$PWD/$npm_package_directories_testnodenv/bin:$PATH bats ${CI:+--tap} test",
+    "posttest": "npm run lint",
     "publish:brew": "brew-publish $npm_package_name v$npm_package_version",
     "postversion": "git push --follow-tags && npm run publish:brew"
   },

test/test_helper.bash

@@ -1,3 +1,5 @@
+# shellcheck shell=bash
+
 EXAMPLE_PACKAGE_DIR="$BATS_TMPDIR/example_package"
 TEST_BASENAME="$(basename "$BATS_TEST_DIRNAME")"
 # TODO: Should this just be $(dirname ...) ?