Skip to content

Re-apply schema language-tag case tests (#25, reverted in #72) - #88

Open
ericprud wants to merge 2 commits into
mainfrom
LanguageTag-schema-case-2
Open

ericprud wants to merge 2 commits into
mainfrom
LanguageTag-schema-case-2

Conversation

@ericprud

Copy link
Copy Markdown
Contributor

Re-applies #25 (reverted in #72) as a single revert-of-the-revert on current main, so the change can be reviewed as one diff. Addresses shexSpec/shex#71.

  • ShExJ keeps the authored case of language tags in schemas (e.g. "en-UK", 1val1LANGTAG_U_L/_L_U).
  • New validation tests: mixed-case tags and language stems match case-insensitively (1val1LANGTAG_*, 1val1languageStemMinuslanguage{,Stem}3_passLAtFR, 1val1languageStemMinuslanguageStemU3_*), with failure cases for non-matching tags.
  • #1val1LANGTAG_pass is renamed #1val1LANGTAG_passLabLTen-fr.

Merge order

Depends on shexjs/shex.js#497, which carries the .val reference results and the parser/validator change. Merge that first. With it, shex.js's full suite passes against this branch.

🤖 Generated with Claude Code

Eric Prud'hommeaux and others added 2 commits September 24, 2026 11:59
This reverts commit 917aba4.

Re-applies #25 (reverted in #72) so the LanguageTag schema-case changes can be revisited.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
ericprud pushed a commit to shexjs/shex.js that referenced this pull request Sep 24, 2026
…Test#88 merges

Point back to #main once it has.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

This branch has not been deployed

No deployments
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