Skip to content

docs: categorize rejected xerial tests in the manifest - #178

Merged
magnusp merged 1 commit into
docs/169-implementation-bugsfrom
docs/172-manifest-categories
Oct 24, 2026
Merged

magnusp merged 1 commit into
docs/169-implementation-bugsfrom
docs/172-manifest-categories

Conversation

@magnusp

@magnusp magnusp commented Oct 23, 2026

Copy link
Copy Markdown
Owner

Why

Rejected xerial tests were recorded without a category, so it was unclear which rejections were permanent and which could be revisited.

What changed

  • Added category annotations to rejected-test entries in upstream-xerial_sqlite-jdbc.manifest.

Scope

  • Conformance selection documentation only. No vendored file or checksum was changed.

Tradeoffs

  • None.

Blast Radius

  • Conformance tooling reads this manifest at build time. Categories are comment lines; the export and verify parsers ignore comment lines except divergence rationale.

Verification

  • ./tools/export-upstream --verify: Verified the vendored tree against the manifest.
  • ./tools/export-upstream --self-test: Self-test passed.

@magnusp
magnusp changed the base branch from main to docs/169-implementation-bugs October 23, 2026 23:32
@magnusp
magnusp added this pull request to stack #183 October 23, 2026 23:32
@magnusp
magnusp force-pushed the docs/172-manifest-categories branch from 0d735a0 to ef08c4d Compare October 23, 2026 23:35
@magnusp
magnusp removed this pull request from stack #183 October 23, 2026 23:39
@magnusp
magnusp force-pushed the docs/172-manifest-categories branch from ef08c4d to 9a82792 Compare October 23, 2026 23:47
@magnusp
magnusp added this pull request to stack #185 October 23, 2026 23:47
@magnusp
magnusp force-pushed the docs/172-manifest-categories branch from 9a82792 to 8fc3571 Compare October 24, 2026 00:08
@magnusp
magnusp merged commit 7d85bb5 into main Oct 24, 2026
10 checks passed
@magnusp
magnusp deleted the docs/172-manifest-categories branch October 24, 2026 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant