Skip to content

ci: fix detect-changes fallback for workflow_dispatch#1375

Merged
steilerDev merged 1 commit intobetafrom
ci/fix-detect-changes-workflow-dispatch
Apr 28, 2026
Merged

ci: fix detect-changes fallback for workflow_dispatch#1375
steilerDev merged 1 commit intobetafrom
ci/fix-detect-changes-workflow-dispatch

Conversation

@steilerDev
Copy link
Copy Markdown
Owner

Summary

  • Fixes Detect Changes step failing with exit code 128 when triggered via workflow_dispatch
  • github.base_ref is only set for pull_request events; defaults to main when empty
  • Allows workflow_dispatch runs to properly detect changed paths and run appropriate jobs

Context

When workflow_dispatch is used to manually trigger CI (e.g., after a [skip ci] auto-fix commit), the Detect Changes job fails because github.base_ref is not set. This causes all downstream jobs to be skipped, resulting in Quality Gates and E2E Gates passing vacuously — which blocks promotion PRs from properly establishing CI status.

Test plan

  • Workflow_dispatch on beta with this fix should run Detect Changes successfully
  • All other CI jobs should run normally after Detect Changes passes

When triggered via workflow_dispatch, github.base_ref is empty,
causing the git diff to fail with exit code 128. Default to main
when base_ref is not set so manual dispatches work correctly.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@steilerDev steilerDev merged commit edb6d2b into beta Apr 28, 2026
12 checks passed
@steilerDev steilerDev deleted the ci/fix-detect-changes-workflow-dispatch branch April 28, 2026 08:36
@github-actions
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.4.2-beta.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.4.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant