Skip to content

fix(ci): expand custodian-audit push trigger to all branches - #296

Merged
ProtocolWarden merged 1 commit into
mainfrom
oc-watchdog/20260614-1637-audit-push-trigger-v2
Jun 14, 2026
Merged

fix(ci): expand custodian-audit push trigger to all branches#296
ProtocolWarden merged 1 commit into
mainfrom
oc-watchdog/20260614-1637-audit-push-trigger-v2

Conversation

@ProtocolWarden

Copy link
Copy Markdown
Owner

Summary

  • Expand custodian-audit push trigger from branches: [main] to branches: ["**"]

Root cause

GitHub does not send pull_request:synchronize events for CONFLICTING PRs. Without push: branches: ["**"], the audit check never runs when commits are pushed to a conflicting PR branch, leaving the audit check absent from the CI suite. The review watcher's ci_wait loop stalls indefinitely waiting for a check that will never appear.

Why this matters

Change

Single-line YAML change — no functional code change.

Supersedes PR #294 (which carried unrelated observer-CLI history causing merge conflicts).

🤖 Generated with Claude Code

GitHub does not send pull_request:synchronize events for CONFLICTING PRs.
Without push: branches: ["**"], the audit check never runs for PR branch
pushes, leaving "audit" absent from the check suite and blocking the review
watcher's ci_wait loop indefinitely.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ProtocolWarden
ProtocolWarden merged commit 2f2f8b9 into main Jun 14, 2026
18 checks passed
@ProtocolWarden
ProtocolWarden deleted the oc-watchdog/20260614-1637-audit-push-trigger-v2 branch June 14, 2026 16: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