Skip to content

fix(ci): declare the failing-job list before the branch that can skip it - #5

Merged
github-actions[bot] merged 1 commit into
mainfrom
ci-redbase-init
Aug 7, 2026
Merged

fix(ci): declare the failing-job list before the branch that can skip it#5
github-actions[bot] merged 1 commit into
mainfrom
ci-redbase-init

Conversation

@catomean

@catomean catomean commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Follow-up to the red-base carve-out: it declared the failing-job list inside
the else branch of the "does the base have CI history?" check, but set -u is
on and the merge site always reads it.

So on a base branch with no CI history at all, the sweep took the
"proceeding" path and died:

[auto-merge] no CI history for main — proceeding
auto-merge-sweep.sh: line 249: base_red_jobs: unbound variable
exit 1

That is auto-merge dead for every PR in the repo, not a degraded path. Latent
here because this base does have CI history — but it is one fresh repo away from
being real, and it was introduced by my own change.

Fixed by declaring it once, before the branch that can skip the assignment.
Covered upstream by a regression test that runs the real script against a gh
reporting no CI history.

🤖 Generated with Claude Code

@github-actions
github-actions Bot merged commit bc63d42 into main Aug 7, 2026
1 check passed
@github-actions
github-actions Bot deleted the ci-redbase-init branch August 7, 2026 10:01
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