Skip to content

ci: bump checkout to v5 and add all-checks-passed gate #8

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
jnasbyupgrade wants to merge 2 commits into
base: master
Choose a base branch
from
+27 −2
Open

ci: bump checkout to v5 and add all-checks-passed gate #8

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
080bfa2
ci: bump actions/checkout to v5 and add all-checks-passed gate
jnasbyupgrade Jul 21, 2026
7853964
ci: scope push trigger to master to avoid double-runs
jnasbyupgrade Jul 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 27 additions & 2 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,13 @@
name: CI
on: [push, pull_request]
# Scope push to master only. With a bare `on: [push, pull_request]`, a commit to
# a branch that has an open PR triggers CI twice (once for push, once for
# pull_request) for the same SHA -- wasteful, and a flaky run can show red next
# to an identical green one. PR branches now run only via pull_request; master
# (post-merge) runs via push.
on:
push:
branches: [master]
pull_request:
jobs:
test:
strategy:
Expand All @@ -12,6 +20,23 @@ jobs:
- name: Start PostgreSQL ${{ matrix.pg }}
run: pg-start ${{ matrix.pg }}
- name: Check out the repo
uses: actions/checkout@v4
uses: actions/checkout@v5
- name: Test on PostgreSQL ${{ matrix.pg }}
run: pg-build-test

# A single stable check name for use as a required status check in branch
# protection. Matrix jobs produce names like "🐘 PostgreSQL 14" that change
# with the matrix; this aggregates them into one. It passes if every needed
# job succeeded or was skipped (e.g. a docs-only push with paths-ignore) and
# fails if any failed or were cancelled.
all-checks-passed:
needs: [test]
if: always()
runs-on: ubuntu-latest
steps:
- name: Check all jobs passed or were skipped
run: |
if [[ "${{ contains(needs.*.result, 'failure') || contains(needs.*.result, 'cancelled') }}" == "true" ]]; then
echo "One or more jobs failed or were cancelled"
exit 1
fi
Loading