Skip to content

fix(ci): release gate no longer fails when a merge cancels main's CI run - #251

Merged
badcuban merged 1 commit into
mainfrom
threadlines/fix-nightly-desktop-release
Sep 6, 2026
Merged

fix(ci): release gate no longer fails when a merge cancels main's CI run#251
badcuban merged 1 commit into
mainfrom
threadlines/fix-nightly-desktop-release

Conversation

@badcuban

@badcuban badcuban commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Problem

Dispatching a desktop release while merges are landing fails at "Verify required CI checks passed". CI cancels the in-progress main run whenever the next merge lands, and the gate treated that cancelled run as the verdict for the release commit. The merge queue had already run the same checks on that exact commit and passed, but the gate ignored that result. This is what broke the manual nightly on 2026-09-06 (run 34024003529): PR #248 merged two minutes after the release started and cancelled PR #250's main CI run.

Fix

scripts/verify-release-ci.ts now treats a cancelled run as "no verdict" and falls through to the newest run of the same check that has one. In-progress runs are still waited on, and a check whose only run was cancelled still fails. The entry point is guarded with import.meta.main so the evaluator can be imported by tests; three focused tests cover the three cases. Running the fixed script against the failed commit passes.

Dispatching a release while merges are landing failed at "Verify required
CI checks passed". CI cancels the in-progress main run whenever the next
merge lands, and the gate treated that cancelled run as the verdict for the
release commit, even though the merge queue had already run the same checks
on that exact commit and passed.

The gate now ignores a cancelled run when the same check has a newer or
older run with a real verdict on the commit. In-progress runs are still
waited on, and a check whose only run was cancelled still fails.
@vercel

vercel Bot commented Sep 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
threadlines Skipped Skipped Sep 6, 2026 3:47pm UTC

Request Review

@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Sep 6, 2026
@badcuban
badcuban enabled auto-merge September 6, 2026 15:47
@github-actions github-actions Bot added the size:S label Sep 6, 2026
@badcuban
badcuban disabled auto-merge September 6, 2026 15:47
@badcuban
badcuban enabled auto-merge September 6, 2026 15:47
@badcuban
badcuban disabled auto-merge September 6, 2026 15:52
@badcuban
badcuban enabled auto-merge September 6, 2026 15:52
@badcuban
badcuban added this pull request to the merge queue Sep 6, 2026
Merged via the queue into main with commit 5b226bd Sep 6, 2026
14 checks passed
@badcuban
badcuban deleted the threadlines/fix-nightly-desktop-release branch September 6, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant