Skip to content

fix(ci): inline cancel-on-failure to fix workflow status shown as skipped#7521

Merged
manusa merged 1 commit intofabric8io:mainfrom
manusa:fix/ci-cancel-on-failure-workflow-status
Mar 5, 2026
Merged

fix(ci): inline cancel-on-failure to fix workflow status shown as skipped#7521
manusa merged 1 commit intofabric8io:mainfrom
manusa:fix/ci-cancel-on-failure-workflow-status

Conversation

@manusa
Copy link
Member

@manusa manusa commented Mar 5, 2026

Summary

  • Fixes the workflow status showing as "skipped" in the GitHub UI when prerequisite checks (style-checks, generate-model) succeed
  • The separate cancel-on-failure job introduced in chore(ci): cancel PR workflows when prerequisite checks fail #7475 gets skipped on success, which causes GitHub to mark the entire workflow as "skipped"
  • Inlines the cancellation as a step within the main job — if: failure() steps are silently skipped without affecting the overall workflow conclusion

…pped

The separate cancel-on-failure job introduced in fabric8io#7475 is skipped when
the main job succeeds, causing GitHub to mark the entire workflow as
"skipped" in the UI. Moving the cancellation into a step within the
main job avoids this since steps with `if: failure()` are silently
skipped without affecting the overall workflow status.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Marc Nuri <marc@marcnuri.com>
@manusa manusa added this to the 7.7.0 milestone Mar 5, 2026 — with automated-tasks
@manusa manusa merged commit 3ba9703 into fabric8io:main Mar 5, 2026
17 of 19 checks passed
@manusa manusa deleted the fix/ci-cancel-on-failure-workflow-status branch March 5, 2026 15:47
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