Add 'npm run lint' to PR build

Discovered in tektoncd#182 (comment) that our JS linter is not being run on PR builds.
Add it to node_test so we can capture these issues and provide faster feedback
to developers without requiring manual review/testing of code style, unused
variables/imports etc.

Also add the node-* folder (used in PR builds) to .gitignore to avoid
issues with unexpected lint config references.

.gitignore

@@ -12,6 +12,7 @@ main
 # dependencies
 /node_modules
 /vendor
+/node-*
 
 # misc
 npm-debug.log
@@ -20,5 +21,3 @@ yarn-error.log
 # System Files
 .DS_Store
 Thumbs.db
-
-

test/presubmit-tests.sh

@@ -72,6 +72,7 @@ function node_test() {
   local failed=0
   echo "Running node tests from $(pwd)"
   node_npm_install || failed=1
+  npm run lint || failed=1
   npm run test:ci || failed=1
   echo ""
   return ${failed}