Skip to content

Commit 2dd617b

Browse files
gpercivacperciva
authored andcommitted
Actions: fix Actions for contributor PRs
- trigger on `pull_request`, not `push`. - don't ignore branch names with a slash in them. We don't use slashes, but external contributors often have branch names with slashes. - we don't need to say anything about `coverity_scan`, because we're not opening PRs for it. Also, fix the case of "runs-on: macOS-14"; it's documented as being lower-case, so if they ever change this to be case-insensitive, the previous version of this file would stop working. (This point courtesy of Claude Sonnet 5.)
1 parent 11e8614 commit 2dd617b

1 file changed

Lines changed: 5 additions & 6 deletions

File tree

.github/workflows/compile.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,10 @@
11
name: Compile & test
22

3-
# Run whenever we push to any branch without a "/", other than [coverity-scan].
3+
# Run whenever there's a new commit in a PR, or if we manually trigger one.
44
on:
5-
push:
6-
branches:
7-
- '*'
8-
- '!coverity-scan'
5+
pull_request:
6+
7+
workflow_dispatch:
98

109
env:
1110
# Shared variables amongst all projects / platforms / compilers.
@@ -68,7 +67,7 @@ jobs:
6867
run: test -z "$(git status --porcelain=v1)"
6968
macOS:
7069
name: macOS
71-
runs-on: macOS-14
70+
runs-on: macos-14
7271
steps:
7372
- name: Checkout code
7473
uses: actions/checkout@v4

0 commit comments

Comments
 (0)