Skip to content

👷 Fix alls-green test dependency#2008

Merged
tiangolo merged 1 commit into
mainfrom
codex/fix-alls-green-test-needs
Jun 21, 2026
Merged

👷 Fix alls-green test dependency#2008
tiangolo merged 1 commit into
mainfrom
codex/fix-alls-green-test-needs

Conversation

@tiangolo

Copy link
Copy Markdown
Member

Summary

  • Add the main test job to the alls-green needs list.
  • Ensure the required aggregate check sees direct test failures, not only downstream coverage status.

Root cause

The alls-green job did not depend directly on the primary test job, so the JSON passed to re-actors/alls-green omitted that job's status.

Validation

  • Parsed .github/workflows/test.yml with PyYAML and asserted the alls-green job needs includes the primary test job.

AI Disclaimer

Codex GPT 5.5, reviewed by hand.

@tiangolo tiangolo marked this pull request as ready for review June 21, 2026 15:39
@tiangolo tiangolo merged commit e9c0418 into main Jun 21, 2026
26 checks passed
@tiangolo tiangolo deleted the codex/fix-alls-green-test-needs branch June 21, 2026 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant