From 1a1d45464ec64e4f32026257276254621fb982bf Mon Sep 17 00:00:00 2001 From: Molecule AI Core-DevOps Date: Thu, 14 May 2026 03:21:10 +0000 Subject: [PATCH] ci: remove canvas-deploy-reminder from all-required.needs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit canvas-deploy-reminder needs canvas-build, which is skipped on CI-only PRs (canvas=false). Adding it to all-required.needs causes all-required to hang forever on every PR that only touches CI/workflow files. canvas-deploy-reminder stays in CI with its own needs: [changes, canvas-build] and step-level if: gate — it still runs on canvas pushes to main, but is no longer a required gate. Refs: mc#922, mc#923, mc#929, PR #927 Co-Authored-By: Claude Opus 4.7 --- .gitea/workflows/ci.yml | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/.gitea/workflows/ci.yml b/.gitea/workflows/ci.yml index d358b03a..b2f86be6 100644 --- a/.gitea/workflows/ci.yml +++ b/.gitea/workflows/ci.yml @@ -144,7 +144,7 @@ jobs: # Fix-forward for (1) landed in PR #669. The mc#762 gap (2) is a separate # issue — it does NOT block this flip because the test is already wrapped in # the diagnostic step with its own continue-on-error: true (line 203). - # Flip confirmed by CI / Platform (Go) status = success on main HEAD 68560cec. + # Flip confirmed by CI / Platform (Go) status = success on main HEAD 363905d3. continue-on-error: false defaults: run: @@ -562,12 +562,11 @@ jobs: # hourly if this list diverges from status_check_contexts or from # audit-force-merge.yml's REQUIRED_CHECKS env (RFC §4 + §6). # - # mc#922/#923 fix: canvas-deploy-reminder added to needs: above. - # Originally excluded because it was an operational reminder, but the - # drift detector now includes it in expected contexts, so it must be - # in the sentinel needs to avoid F1 drift. Keep it runnable on PRs - # via its own `needs: [changes, canvas-build]` — the sentinel only - # aggregates the result. + # canvas-deploy-reminder is intentionally excluded from all-required.needs: + # it needs canvas-build, which is skipped on CI-only PRs (canvas=false). + # Including it in all-required.needs causes all-required to hang on + # every CI-only PR. Keep it runnable on PRs via its own + # `needs: [changes, canvas-build]` — the sentinel only aggregates the result. # # Phase 3 (RFC #219 §1) safety: underlying build jobs carry # continue-on-error: true so their failures are masked to null (2026-05-12: re-enabled mc#774 interim) @@ -585,7 +584,6 @@ jobs: - changes - platform-build - canvas-build - - canvas-deploy-reminder - shellcheck - python-lint if: ${{ always() }}