Repository navigation
ci: report every macOS shard failure on main - #1329
Merged
Merged
Conversation
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Each main-push shard reports all failures in one run. PR shards and the macOS gate keep their existing fail-fast behavior, as do nightly and manual runs. Document the setting in
docs/ci-performance.md.Closes #1325
Evidence
After: the updated push command reported 2/2 failed. Both runs correctly returned exit 100. Rendered PR, nightly and manual commands still reported 1/2 failed. Linux command-level probe used nextest 0.9.149,
hash:1/1, and the repository's default profile settings.PATH=/tmp/thirdshift-1027-tools:$PATH CARGO_BUILD_JOBS=2 NEXTEST_TEST_THREADS=2 scripts/affected-tests mainpassed: 3,011 passed (7 slow), 66 skipped. Nextest execution took 2,688 seconds with two test threads; fresh compilation took 4m 18s.main: Standards 0 findings; Spec 0 findings.Merge Danger
Door: two-way
Revert the workflow flag and documentation to restore the previous behavior.
Blast Radius: CI
Failing main-push macOS shards take longer because all tests finish. Failure status is preserved.
Unaddressed findings
Standards
None.
Spec
None.
Security
None.
Security review outcome
No unaddressed introduced findings.
Review coverage
Both reviewers read every changed file:
.github/workflows/ci.ymlanddocs/ci-performance.md. Unread changed files: none. Generated-output exclusions: none. Final reports and finding counts are saved in.thirdshift-review-HQrOtS/standards.md,spec.mdandsummary.jsonin the session worktree.Built with codex codex-cli 0.164.0 · gpt-6.1-sol · xhigh