From 64cce6c18344103229b8283923b559eab4e5f02c Mon Sep 17 00:00:00 2001 From: Molecule AI Core-DevOps Date: Thu, 14 May 2026 01:43:57 +0000 Subject: [PATCH] fix(ci): add canvas-deploy-reminder to staging all-required.needs (mc#923) mc#923 ci-drift root fix for staging branch. canvas-deploy-reminder exists in staging ci.yml. Although the job is gated by `if: github.event_name == 'push' ...` and ci_job_names() should exclude it from F1 drift, the drift detector is flagging it. Apply the same fix as mc#922 for main: add to all-required.needs:. Co-Authored-By: Claude Opus 4.7 --- .gitea/workflows/ci.yml | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/.gitea/workflows/ci.yml b/.gitea/workflows/ci.yml index cad7a727..7e779f02 100644 --- a/.gitea/workflows/ci.yml +++ b/.gitea/workflows/ci.yml @@ -535,11 +535,13 @@ jobs: # hourly if this list diverges from status_check_contexts or from # audit-force-merge.yml's REQUIRED_CHECKS env (RFC §4 + §6). # - # Excluded from `needs:`: `canvas-deploy-reminder` — gated by - # `if: ... github.event_name == 'push' && github.ref == 'refs/heads/main'`, - # so on PR events it's legitimately `skipped`. The drift detector - # explicitly excludes `github.event_name`-gated jobs from F1 (see - # `.gitea/scripts/ci-required-drift.py::ci_job_names`). + # mc#923 fix: canvas-deploy-reminder added to needs: above. + # The job's `if:` gate (push-to-main only) means it is legitimately + # skipped on PRs — the drift detector's F1 should exclude it (it uses + # ci_job_names() which skips github.event_name-gated jobs), but + # to be safe and consistent with main, include it in needs:. The + # all-required sentinel will see it as 'skipped' on PRs and handle + # that per its Phase-3 exclusion logic. # # 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) @@ -557,6 +559,7 @@ jobs: - changes - platform-build - canvas-build + - canvas-deploy-reminder - shellcheck - python-lint if: always()