chore: move the JS validator to OO-LD/oold-js - #156
Merged
Conversation
The specification repository no longer carries an implementation: the
corpus and the code that checks it are versioned separately, and the
Node dependency goes with it.
- scripts/{validate,schema_to_frame,pattern_lint}.mjs removed
- Makefile install and validate-reference targets dropped
- package.json keeps only the legacy-keyword extractor, which uses
nothing outside Node's standard library
Closes #91
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.
Closes #91. Pairs with OO-LD/oold-python#144, which repoints the parity suite at the new home.
The three validator files are now OO-LD/oold-js, extracted unchanged and tagged
v0.1.0. Verified behaviour-preserving: run over this repository'sexamples/it reports150/150 checks passed, 1 warning, byte-identical tomake validate-referencebefore the move.The one change the move required is where the meta-schemas come from. They were read relative to the script's own location, which does not survive leaving this repository, so
oold-jstakes--meta <dir>instead. It vendors no meta-schemas: they are owned here and versioned with the specification, so pinning them is the caller's decision.Removed
scripts/validate.mjs,scripts/schema_to_frame.mjs,scripts/pattern_lint.mjsmake installandmake validate-referencepackage.json'sbin,dependenciesandvalidatescript, leaving onlyextract-legacy, which the migration docs link to and which imports nothing outside Node's standard libraryThis repository now has no Node dependencies at all.
Docs
docs/tooling.mdadvertisednpx --yes github:OO-LD/oold-schema oold-validate <dir>as the way to check your own schemas. Thatbinis gone; the equivalent is nowuvx --from "oold[validation]" oold validate <dir>, which is the maintained implementation and the one this repository's own CI runs.README.mdanddocs/introduction.mdno longer tell a reader tonpm install.Verification
make validate:examples380 ok / 0 failed,examples/compliance72 ok / 0 failed across 54 targets.make checkexit 0, 80 rules, no drift.