chore: bump spec submodule (ASCII heading-ids corpus + new examples) - #15
Merged
Conversation
- spec submodule bumped to markup-carve/carve#16 (Adopt ASCII-safe heading id slugs spec + corpus + re-vendor). The corpus fixture for 19-heading-ids now matches the ASCII slugs the impl emits. - Un-skips '19-heading-ids' in IMPLEMENTED — the TODO PR #13 left pending the corpus bump. - Adds the 17 new corpus entries (25-41) the spec now ships: table doubled alignment marker, fenced-code shorter inner fence, blockquote caption after blank line, escaped/code-span pipes in table cells, abbreviation/mention/tag word-boundary rules, stacked rowspan, reference-link / collapsed / unresolved cases, three smart-typography splits, multi-line cell continuation, and rowspan with multi-line content. All 17 pass byte-for-byte against the current impl. - Adds vitest.config.ts to scope test discovery to test/**/*.test.ts. The bumped spec submodule carries spec/tests/*.test.mjs files (node:test runners for the carve repo's own CI) that vitest would otherwise try to load and fail on as 'no test suite found'. Suite: 217/217 (was 187). Typecheck clean.
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.
Follow-up to #13 + companion to markup-carve/carve#16 (now merged).
What's in it
specsubmodule bumped to markup-carve/carve@0e05cc3 — picks up the ASCII-slug corpus fixture for19-heading-ids(café-notes→cafe-notes,über-uns→uber-uns, plus the matching</#cafe-notes>crossref) and 17 newly-added corpus examples.test/corpus.test.tsun-skips19-heading-ids(the TODO comment ASCII-safe heading id slugs (baked Unicode -> ASCII translit) #13 left behind) and adds the new 25–41 entries toIMPLEMENTED. All 17 new entries pass byte-for-byte against the current impl — no impl changes needed:|inside cell\brule@mention boundary#tag boundary^+-continuation rowvitest.config.ts(new, 12 lines) — narrowsvitest's test discovery totest/**/*.test.ts. The bumped spec submodule carries its ownspec/tests/*.test.mjsfiles (the carve repo'snode:testrunners for spec-corpus conformance + normativity checks), whichvitestwould otherwise try to load and fail on as "no test suite found".Suite
217/217 (up from 187 before the bump). Typecheck clean.
Closes out
test/corpus.test.tsskip annotation