From ee27c41e834938db8213fbb34199247d2e8b6828 Mon Sep 17 00:00:00 2001 From: ScriptedAlchemy Date: Fri, 4 Sep 2026 21:12:22 +0000 Subject: [PATCH] ci: fix the stale Verify required-context comment after the merge-queue readiness audit The Verify matrix comment in ci.yml still named "Verify (Node 24)" as the required check on main; #549 replaced it with the static "Verify gate" job. Audit of every required-check workflow for merge_group readiness (#536) found no other gap; findings and the settings commands live in the PR body. --- .github/workflows/ci.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index c19d5d394..294d9fed1 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -121,9 +121,9 @@ jobs: runs-on: ubuntu-latest timeout-minutes: 45 # Measured PR Verify cost was 3 legs × ~7 minutes, about 75% of PR - # runner-minutes. PRs and merge-queue entries run newest-LTS Node 24 (the - # "Verify (Node 24)" context is the required check on main); every main - # push and manual dispatch still gates on the full supported matrix. + # runner-minutes. PRs and merge-queue entries run newest-LTS Node 24 + # (main requires the static "Verify gate" job below, never a leg name); + # every main push and manual dispatch still gates on the full matrix. strategy: fail-fast: false matrix: