Skip to content

ci: add gate checks that report on docs-only PRs and name the Windows job - #2775

Open
dunglas wants to merge 1 commit into
mainfrom
ci/required-checks
Open

dunglas wants to merge 1 commit into
mainfrom
ci/required-checks

Conversation

@dunglas

@dunglas dunglas commented Oct 26, 2026

Copy link
Copy Markdown
Member

Prepares required status checks on main.

  • Tests passed and Sanitizers passed aggregate their workflow's jobs. Skipped jobs count as passing, failed or cancelled ones fail the gate.
  • Docs-only PRs used to skip these workflows entirely through paths-ignore, so a required check would stay pending forever. A Detect changes job now skips the jobs instead, and the gate still reports.
  • The PHP 8.6 job keeps continue-on-error, so php-src master breakage doesn't block merges.
  • The Windows job is renamed Windows build: build also exists in the Docker workflow, where it is skipped on PRs and could satisfy a required check.

Once merged, the ruleset can require Lint Code Base, Tests passed, Sanitizers passed and Windows build.

This branch has not been deployed

No deployments
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