ci(staging): remove canvas-deploy-reminder from all-required.needs (deadlock fix) #944

Closed
core-devops wants to merge 1 commits from fix/staging-canvas-reminder-deadlock into staging

View File

@ -535,13 +535,9 @@ jobs:
# hourly if this list diverges from status_check_contexts or from
# audit-force-merge.yml's REQUIRED_CHECKS env (RFC §4 + §6).
#
# 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.
# 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.
#
# 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)
@ -559,7 +555,7 @@ jobs:
- changes
- platform-build
- canvas-build
- canvas-deploy-reminder
# - canvas-deploy-reminder ← intentionally excluded — see deadlock rationale above
- shellcheck
- python-lint
if: always()