Skip to content

fix(ci): backport #926 — move matrix.name guard to step-level if - #927

Merged
sabbour merged 1 commit into
devfrom
backport/926-publish-images-fix
Aug 25, 2026
Merged

fix(ci): backport #926 — move matrix.name guard to step-level if#927
sabbour merged 1 commit into
devfrom
backport/926-publish-images-fix

Conversation

@sabbour

@sabbour sabbour commented Aug 25, 2026

Copy link
Copy Markdown
Owner

Backport of #926 to \dev.

Fixes \publish-images.yml\ workflow parse failure caused by \matrix.name\ in a job-level \if\ condition (not allowed by GitHub Actions). GHCR image publishing was silently broken for all events.

)

job-level \if\ conditions do not have access to the \matrix\ context in
GitHub Actions — using \matrix.name\ there causes a workflow parse failure
and the entire workflow silently fails with zero jobs.

Fix by:
- Removing the job-level \if\ condition that referenced \matrix.name\
- Moving the path-filter guard to a step-level \if\ on the build step,
  where \matrix\ context is valid
- Making the \changes\ job always run (condition moved to its steps) so
  that \�uild\, which needs it, is never skipped due to a skipped dep
- Using \github.event_name != 'push' || ref == main || changes.output\
  logic instead of the equivalent but invalid job-level expression

This unblocks GHCR image publishing for all triggers.

Co-authored-by: Cypher <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 11857092-1327-41bd-8708-b2ca85e674dd
@sabbour
sabbour merged commit 059098b into dev Aug 25, 2026
13 checks passed
@sabbour
sabbour deleted the backport/926-publish-images-fix branch August 25, 2026 17:27
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