Repository navigation
chore: shorten CI with independent nextest test shards - #218
Merged
Merged
Conversation
axisrow
marked this pull request as ready for review
October 24, 2026 10:02
Blogbotana
approved these changes
Oct 24, 2026
axisrow
referenced
this pull request
in axisrow/BeamCraft
Oct 24, 2026
…con (#204), rigid member placement (#222), D03.3 unique versions (#220) These upstream commits re-land fork work adapted in the upstream review round: the nextest CI shards, the enamel macOS icon with its sips-only pipeline, rigid member placement, and the D03.3 drawing lifecycle with unique published versions. The upstream copies win where they re-land fork work (the benchmark workflow, the cli README, the D03.3 store and drawing tests, the macOS icon pipeline in development.md and generate-icon.sh, the ATTRIBUTION icon rows); the fork keeps its side where it is richer or fork-only: the help text shape in main.rs, the Windows/Linux icon rows and sections (the py generator stays for them), and plate.rs where the auto-merge had duplicated our own inspect_manifest_assets and summarize_manifest. The local triple is green on the merged tree. Co-Authored-By: Claude Code <[email protected]>
This was referenced Oct 25, 2026
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.
What this changes
Runs workspace tests on Windows, Ubuntu and macOS using one nextest archive build and eight independent test shards per OS. Each platform starts its shards as soon as its own archive is ready. Clippy, fixture verification and module boundary checks run separately on all three platforms; format and documentation generation remain Linux gates. Doctests run on every platform. Archive uploads disable redundant compression and extraction preserves target/tmp and compile-time executable paths.
Adds a manual Ubuntu benchmark comparing in-place nextest with four/eight shards. Cold runs use an isolated cache prefix; warmup runs are excluded; warm samples require an exact cache hit.
The branch starts at upstream master beeedb4 and changes only two workflow files. Corresponding changes are merged in the fork via axisrow#125 and axisrow#126.
How you verified it
Never-crash
CI configuration only; no application code, unsafe code or new panic paths.
Provenance
No new assets or copied commercial/GPL/AGPL code. No attribution changes required.
Anything a reviewer should look at first
Twenty-four test shards trade aggregate runner time for elapsed time. Preserve all platform doctest gates and archive extraction layouts. For each benchmark variant, run cold once, warmup once (excluded), then three warm runs on the same ref/revision. A warm miss invalidates the sample; complete warmup before retrying. The benchmark workflow must exist on the default branch before manual dispatch.
Verified final integration status
Earlier preparation/draft notes in this historical description are superseded by GitHub's final state: merged into master on 2026-10-24T14:51:14Z. Independently fetched integration commit ae5ff46: chore: shorten CI with independent nextest test shards (#218). This records delivery of this PR's limited slice, not completion of open acceptance gates.
Issue tracking
Fixes #209