chore: bump spec corpus to carve 95fe6e8 - #1305
Merged
Merged
Conversation
dereuromark
changed the title
dereuromark
force-pushed
the
automation/bump-spec
branch
2 times, most recently
from
August 16, 2026 10:23
1c58a18 to
1ec8bd3
Compare
dereuromark
force-pushed
the
automation/bump-spec
branch
from
August 16, 2026 11:44
1ec8bd3 to
e885be8
Compare
dereuromark
marked this pull request as ready for review
August 16, 2026 11:45
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
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
1 participant
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.
Automated bump of the
tests/specsubmodule to carve main (95fe6e8), rebased onto currentmain.The two spec commits this picks up are
spec: rule the two-caption collision the wrapper clause left openandtest: pin the cell forms the three writers disagreed about, and move the engine pin.Why it was red
Nine
CarveCorpusTestcases failed, all of them318-composite-figures-*, and every diff had the same shape: the corpus expected the group's panels to nest directly under the<figure>while the engine still wrapped them inThat is not a corpus mismatch this branch introduced. It is a stale base. The bump branch forked from
35f2404, and the engine-side change that drops the wrapper landed onmainas #1306 after this branch's checks had already run. The checks described a tree that no longer existed.Rebasing onto
main(e0e7cbc, which contains #1306) resolves the gitlink to95fe6e8and the corpus passes with no allowlist entry added and no skip list widened. The submodule bump is still the only change in the diff; the bot's commit subject claimed an implementation attempt that was never in the tree, so the subject now says only what the commit does.