Skip to content

馃懛 Fix alls-green test dependency - #1848

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

馃懛 Fix alls-green test dependency#1848
tiangolo merged 1 commit into
masterfrom
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.

@github-actions github-actions Bot added the internal Internal changes label Jun 21, 2026
@tiangolo
tiangolo marked this pull request as ready for review June 21, 2026 15:39
@tiangolo
tiangolo merged commit 6ff98d6 into master Jun 21, 2026
25 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

internal Internal changes

1 participant