Only one job for browser tests on travis (josdejong#1594)

* only one job for browser tests on travis

Only run local tests in pull requests and only run browser tests
when running in a branch.
Should speed up ci runs slightly.

* let travis do default caching

See: https://docs.travis-ci.com/user/languages/javascript-with-nodejs/#caching-with-npm

* do not do anything fancy with git

Simpler config is better provided it does not slow down ci.

* Revert "do not do anything fancy with git" - to time ci

This reverts commit 543dc3a.

* Revert "Revert "do not do anything fancy with git" - to time ci" -
enough of this hokie cokie

This reverts commit 718c351.

Author: harrysarson

.travis.yml

@@ -12,21 +12,12 @@ jobs:
       script: npm run lint
       node_js: lts/*
       name: Lint
-    - script: npm run test:browser
+    - script: 'if [ "$TRAVIS_PULL_REQUEST" = "false" ]; then npm run test:browserstack; else npm run test:browser; fi'
       node_js: lts/*
-      name: Local Browser Test
-    - script: 'if [ "$TRAVIS_PULL_REQUEST" = "false" ]; then npm run test:browserstack; else true; fi'
-      node_js: lts/*
-      name: Browserstack Test
+      name: Browser Test
     - script: npm run build-and-test
       node_js: lts/*
       name: Build Test
     - script: npm run coverage && npx codecov
       node_js: lts/*
       name: Coverage
-git:
-  depth: 5
-
-cache:
-  directories:
-    - node_modules