Skip to content

ci: use step-level conditionals so matrix check names are always reported - #753

Closed
skrawcz wants to merge 1 commit into
mainfrom
ci/fix-required-checks-skip
Closed

skrawcz wants to merge 1 commit into
mainfrom
ci/fix-required-checks-skip

Conversation

@skrawcz

@skrawcz skrawcz commented May 3, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #752. When a job is skipped via job-level if:, GitHub reports a single 'skipping' entry without matrix suffixes. This means required checks like Release Validation / install-and-smoke (3.12) are never reported, blocking merge.

Fix: Remove job-level if: and instead gate each step with if: needs.check-paths.outputs.should_run == 'true'. The jobs always run (reporting their full matrix names as passing), but exit immediately when only docs/website files changed.

This unblocks PRs #738, #740, #730, #737.

…rted

When a job is skipped via job-level `if:`, GitHub reports a single
'skipping' entry without matrix suffixes. This means required checks
like 'Release Validation / install-and-smoke (3.12)' are never reported,
blocking merge.

Fix: remove job-level `if:` and instead gate each step. The jobs always
run (reporting their full matrix names), but exit immediately when only
docs/website files changed.
@skrawcz skrawcz added the area/ci Workflows, build, release scripts label May 3, 2026
@skrawcz
skrawcz requested a review from elijahbenizzy May 3, 2026 06:09
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

This PR has had no activity for 91 days.
Closing to keep the PR list manageable.

Feel free to reopen when you are ready to continue.
If the branch has conflicts, a fresh PR against main might be easier.

@github-actions github-actions Bot closed this Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci Workflows, build, release scripts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants