fix(ci): add canvas-deploy-reminder to staging all-required.needs (mc#923)
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 22s
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 24s
CI / Detect changes (pull_request) Successful in 1m7s
E2E API Smoke Test / detect-changes (pull_request) Successful in 1m6s
Handlers Postgres Integration / detect-changes (pull_request) Successful in 1m4s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 15s
gate-check-v3 / gate-check (pull_request) Successful in 25s
qa-review / approved (pull_request) Successful in 16s
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 49s
security-review / approved (pull_request) Successful in 17s
sop-checklist-gate / gate (pull_request) Successful in 20s
lint-continue-on-error-tracking / lint-continue-on-error-tracking (pull_request) Failing after 2m14s
sop-tier-check / tier-check (pull_request) Successful in 18s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m30s
Lint workflow YAML (Gitea-1.22.6-hostile shapes) / Lint workflow YAML for Gitea-1.22.6-hostile shapes (pull_request) Successful in 1m41s
lint-mask-pr-atomicity / lint-mask-pr-atomicity (pull_request) Successful in 2m9s
Lint pre-flip continue-on-error / Verify continue-on-error flips have run-log proof (pull_request) Successful in 2m5s
lint-required-context-exists-in-bp / lint-required-context-exists-in-bp (pull_request) Successful in 2m17s
CI / Shellcheck (E2E scripts) (pull_request) Successful in 32s
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 21s
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 11s
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 9s
CI / Python Lint & Test (pull_request) Failing after 2m6s
CI / Platform (Go) (pull_request) Failing after 5m48s
CI / Canvas (Next.js) (pull_request) Failing after 13m38s
CI / Canvas Deploy Reminder (pull_request) Has been skipped
CI / all-required (pull_request) Failing after 3s
sop-checklist / na-declarations (pull_request) awaiting /sop-n/a declaration for: qa-review, security-review
sop-checklist / all-items-acked (pull_request) [info tier:low] acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: comprehensive-testing, l
audit-force-merge / audit (pull_request) Has been skipped
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 22s
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 24s
CI / Detect changes (pull_request) Successful in 1m7s
E2E API Smoke Test / detect-changes (pull_request) Successful in 1m6s
Handlers Postgres Integration / detect-changes (pull_request) Successful in 1m4s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 15s
gate-check-v3 / gate-check (pull_request) Successful in 25s
qa-review / approved (pull_request) Successful in 16s
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 49s
security-review / approved (pull_request) Successful in 17s
sop-checklist-gate / gate (pull_request) Successful in 20s
lint-continue-on-error-tracking / lint-continue-on-error-tracking (pull_request) Failing after 2m14s
sop-tier-check / tier-check (pull_request) Successful in 18s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m30s
Lint workflow YAML (Gitea-1.22.6-hostile shapes) / Lint workflow YAML for Gitea-1.22.6-hostile shapes (pull_request) Successful in 1m41s
lint-mask-pr-atomicity / lint-mask-pr-atomicity (pull_request) Successful in 2m9s
Lint pre-flip continue-on-error / Verify continue-on-error flips have run-log proof (pull_request) Successful in 2m5s
lint-required-context-exists-in-bp / lint-required-context-exists-in-bp (pull_request) Successful in 2m17s
CI / Shellcheck (E2E scripts) (pull_request) Successful in 32s
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 21s
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 11s
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 9s
CI / Python Lint & Test (pull_request) Failing after 2m6s
CI / Platform (Go) (pull_request) Failing after 5m48s
CI / Canvas (Next.js) (pull_request) Failing after 13m38s
CI / Canvas Deploy Reminder (pull_request) Has been skipped
CI / all-required (pull_request) Failing after 3s
sop-checklist / na-declarations (pull_request) awaiting /sop-n/a declaration for: qa-review, security-review
sop-checklist / all-items-acked (pull_request) [info tier:low] acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: comprehensive-testing, l
audit-force-merge / audit (pull_request) Has been skipped
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 <noreply@anthropic.com>
This commit is contained in:
parent
3a30b07309
commit
4e886c91ae
@ -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()
|
||||
|
||||
Loading…
Reference in New Issue
Block a user