perf(spec-tools,ci): parallel PR spec-doc builds, publish-only social cards - #3101
Merged
danceratopz merged 4 commits intoJul 6, 2026
Merged
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## forks/amsterdam #3101 +/- ##
===================================================
+ Coverage 93.23% 93.30% +0.06%
===================================================
Files 624 624
Lines 36924 36986 +62
Branches 3373 3383 +10
===================================================
+ Hits 34427 34508 +81
+ Misses 1704 1695 -9
+ Partials 793 783 -10
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
spencer-tb
force-pushed
the
ci/docs-build-speedup
branch
from
July 3, 2026 16:52
52d66d1 to
8c8f2a3
Compare
spencer-tb
changed the title
spencer-tb
force-pushed
the
ci/docs-build-speedup
branch
from
July 3, 2026 17:14
8c8f2a3 to
653c646
Compare
spencer-tb
added
C-feat
spencer-tb
changed the title
spencer-tb
marked this pull request as ready for review
July 3, 2026 17:25
danceratopz
self-requested a review
July 6, 2026 10:06
Open
4 tasks
danceratopz
reviewed
Jul 6, 2026
danceratopz
left a comment
danceratopz
left a comment
Member
There was a problem hiding this comment.
Thanks for tackling this. Adds quite some complexity, but it really does help shorten runtimes. Few comments below. I made one suggestion directly in:
danceratopz
changed the title
Contributor
Author
|
Merged your PR in! Thx |
danceratopz
reviewed
Jul 6, 2026
danceratopz
left a comment
danceratopz
left a comment
Member
There was a problem hiding this comment.
There's one clean-up required that was missed in spencer-tb#24. Let me commit that real quick.
danceratopz
reviewed
Jul 6, 2026
The `docc_shards` unit test never ran in CI: `fill` and the docs build skip `tests/docc` via `_add_default_ignores`, and no other job targets it, so the `compute_shards` coverage and overlap assertions went unchecked. The module itself runs via `docs-spec-parallel`, but its invariants were never asserted. Move the test to `tests/evm_tools/`, which the `spec-tools` job runs, so the assertions now execute in CI. That directory is already in `fill`'s default ignores, so the separate `tests/docc` entry is removed.
danceratopz
approved these changes
Jul 6, 2026
danceratopz
left a comment
danceratopz
left a comment
Member
There was a problem hiding this comment.
Thanks for this!
Confirmed that the last commit does actually include these tests in the spec-tools CI. All good!
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
2 participants
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.
🗒️ Description
Aims to improve per PR docs CI workflows.
Spec doc builds now run as four parallel docc shards over consecutive fork sets cutting
Build Spec Docsto ~4m.MkDocs social cards are now publish only, trimming
Build HTML Docsto ~5mins.Note, forward / cross-shard references are validated only by the serial default-branch build, and optionally log what PruneReferencesTransform prunes so the coverage boundary is visible in CI, more info in #3101 (comment).
🔗 Related Issues or PRs
N/A.
✅ Checklist
just static<type>(<area>):, where<type>and<area>come from an approrpriateC-<type>, respectivelyA-<area>, label. The title should match the a target squash commit message.Cute Animal Picture