5f826f6bc7
Some checks failed
CI / lint (push) Failing after 59s
Deploy / notebooks (push) Has been skipped
Deploy / docs (push) Has been skipped
Deploy / report (push) Successful in 16s
Deploy / zotero (push) Has been skipped
Deploy / api (push) Has been skipped
Deploy / mc (push) Has been skipped
CI / test (push) Successful in 33m10s
The Report step in the deploy workflow's report job was running unconditionally (only the JOB had `if: always() && !cancelled()`, not the step), so api.diag.ci fired on every successful Deploy run and auto-filed a fake "ci/Deploy/report failed" issue. Closes #404, #405, #406 (false positives). Add `if: contains(needs.*.result, 'failure')` to the Report step. Matches the `if: failure()` pattern already on every other workflow's report step (ci.yml, harden.yml, infra-ci.yml, etc). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>