Update GitHub Actions CI to include pull requests (#6705)

* Update GitHub Actions CI to include pull requests

Update GitHub Actions CI to include pull requests, and limit both triggers to the master branch.

* Run coveralls publish on bash

Always run coveralls publish on bash - on Windows, the default shell is PowerShell but that is not appropriate for this workflow.

* Run GitHub Actions against the 2.9 branch
* Temporarily remove macOS continuous integration

Chrome is unhappy executing the tests after an update to macOS Catalina.
Remove macOS CI until that can be resolved.

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index ca7f1caab94aa3192025a2ddafa0060416a05d92..968f66f842ce70beceb602bf6f1952bcbcfcb03b 100644 (file)
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -1,6 +1,14 @@
 name: CI
 
-on: [push]
+on:
+  push:
+    branches:
+      - master
+      - "2.9"
+  pull_request:
+    branches:
+      - master
+      - "2.9"
 
 jobs:
   build:
@@ -8,7 +16,7 @@ jobs:
 
     strategy:
       matrix:
-        os: [ubuntu-latest, windows-latest, macos-latest]
+        os: [ubuntu-latest, windows-latest]
       fail-fast: false
 
     steps:
@@ -46,4 +54,5 @@ jobs:
         gulp bower
     - name: Publish Test Results
       run: cat ./coverage/lcov.info | ./node_modules/.bin/coveralls
+      shell: bash
       continue-on-error: true