Skip to content

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

Open
jnasbyupgrade wants to merge 2 commits into
Postgres-Extensions:masterfrom
jnasbyupgrade:ci-hygiene
Open

ci: bump checkout to v5 and add all-checks-passed gate#8
jnasbyupgrade wants to merge 2 commits into
Postgres-Extensions:masterfrom
jnasbyupgrade:ci-hygiene

Conversation

@jnasbyupgrade

@jnasbyupgrade jnasbyupgrade commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Small CI-hygiene changes, independent of the larger test-capabilities work.

  • actions/checkout@v4 → v5 — v4 emits a Node 20 deprecation warning on current runners.
  • all-checks-passed aggregation job — one stable required-status-check name for branch protection, instead of the matrix-dependent 🐘 PostgreSQL <n> names. Passes if every needed job succeeded or was skipped; fails if any failed/cancelled.
  • Scope push trigger to masteron: [push, pull_request] double-runs CI on every commit to a PR branch (once as push, once as pull_request) for the same SHA; a flaky run could then show red next to an identical green one. PR branches now run only via pull_request; master runs post-merge via push.

No change to what's tested (still pg-build-test across the PG 9.3–17 matrix). Follow-up PRs will add the pg_upgrade and pg_tle jobs.

🤖 Generated with Claude Code

- actions/checkout@v4 -> v5 (v4 triggers a Node 20 deprecation warning).
- Add an all-checks-passed aggregation job so branch protection can use one
  stable required check instead of the matrix-dependent per-version names.

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 6d307804-a77d-45f5-9d0a-26895ce54480

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

on: [push, pull_request] double-triggers CI on every commit to a PR branch
(once as push, once as pull_request) for the same SHA. Scope push to master
so PR branches run only via pull_request; master runs post-merge via push.

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant