Skip to content

fix: take latest result of run checks - #627

Merged
dghubble merged 2 commits into
poseidon:mainfrom
efiShtain:main
Jul 19, 2025
Merged

fix: take latest result of run checks#627
dghubble merged 2 commits into
poseidon:mainfrom
efiShtain:main

Conversation

@efiShtain

Copy link
Copy Markdown
Contributor

When a run check failed, next time we run we take it into account, meaning it will always fail the check
This makes sure we take only the latest version of the run check

efiShtain added 2 commits May 24, 2025 13:47
This make sure we only take into account the latest
result of a run, fixing a bug where old failed check runs
that are now passing in latest version still breaks the check
@CLAassistant

CLAassistant commented May 24, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@dghubble
dghubble merged commit 990a77d into poseidon:main Jul 19, 2025
bradh352 added a commit to c-ares/c-ares that referenced this pull request Jul 3, 2026
## Summary

Adds a single **All Checks** aggregator workflow that waits for every
*other* check on the PR commit to pass or be skipped, using
[poseidon/wait-for-status-checks](https://github.com/poseidon/wait-for-status-checks)
(pinned to v0.6.0 + poseidon/wait-for-status-checks#627).

This makes it possible to mark just **All Checks** as the required
status check on the `main` ruleset: any workflow added (or renamed) in
the future is then automatically gated through it — if it fails, this
check fails and blocks the PR — with no per-check ruleset edits ever
needed.

## Notes

- The action ignores itself via `ignore: "All Checks"` (self-ignore
isn't automatic once a job `name` is set).
- Skipped checks (e.g. path-filtered workflows like Codespell) count as
passing.
- Timeout is 3 hours to cover the slower emulated/VM jobs (Solaris,
NetBSD, OpenBSD, QNX).
- After merging, the `main` ruleset's required status checks can be
reduced to just `All Checks`.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants