mirror of
https://github.com/bitcoin/bitcoin.git
synced 2026-02-24 00:06:09 +01:00
eb510f8678ci: fail fast in test-each-commit script (Lőrinc)04c4d71008ci: remove commit count limit from `test-each-commit` (Lőrinc) Pull request description: ### Problem `test-each-commit` currently tests only a limited number of ancestor commits in a PR, so failures introduced deeper in the commit stack might be missed. ### Fix Remove the max-count limit so `test-each-commit` runs the full build + unit + functional test flow on every non-head PR commit, while keeping the PR tip excluded because it is already covered by the normal CI jobs. It will also stop after the first failure to surface the root cause sooner and keep logs readable when testing ancestor commits. ### Examples * Example failure 10 commits deep: https://github.com/l0rinc/bitcoin/actions/runs/21390976651/job/61577575033?pr=105 in https://github.com/l0rinc/bitcoin/pull/105 * Example pass with >7 dummy commits: https://github.com/l0rinc/bitcoin/actions/runs/21392557521/job/61595159841?pr=106 in https://github.com/l0rinc/bitcoin/pull/106 --------- Note: this PR has gone through a few iterations, the latest one just extends the existing job. ACKs for top commit: maflcko: lgtm ACKeb510f8678🕓 hebasto: re-ACKeb510f8678. willcl-ark: ACKeb510f8678Tree-SHA512: 5aadafd32daad610ce882277802c390642dc34f7d5bfa71d4b503ee007942d1ebafce2a3430ea5fd2af6673c83f9aee42450043be4722d7c02407d90920f8bce