From: Edward Thomson <ethomson@edwardthomson.com>
Date: Tue, 3 Dec 2019 22:52:39 +0000 (+1100)
Subject: Update GitHub Actions CI to include pull requests (#6705)
X-Git-Tag: v3.0.0-alpha~208
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f615035926d814fd1cbd9eaa215eef672c63c942;p=thirdparty%2FChart.js.git

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 ca7f1caab..968f66f84 100644
--- 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