CLI: Read JSON instances one to a line from stdin. #211
Conversation
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
|
Hi! Thanks so much for the PR. This certainly seems useful -- as implemented though it breaks backwards compatibility -- maybe it'd be useful to have as a new command line flag (or alternatively, read from stdin if no instances were provided). It'd also be nice to have a test for the new functionality. Does that sound reasonable? |
|
You're welcome. I was thinking a |
|
Implemented this in #213, let me know what you think. |
Julian
added a commit
that referenced
this pull request
Apr 14, 2018
cb2b906 Merge pull request #219 from mohawk2/fix-typo 99452e4 fix typo b242817 Merge pull request #213 from davishmcclurg/ipv6-iri-format 2146b02 Fix invalid IPv6 IRI 71843cb Merge pull request #211 from deBhal/add-complex-selection-tests f415a88 Duplicate complex tests for anyOf,oneOf into draft6 and draft7 7c518b4 add complex types test for oneOf, anyOf efb704f Merge pull request #212 from json-schema-org/update-ajv a8f9f82 fix: ajv test to use id/$id correctly git-subtree-dir: json git-subtree-split: cb2b9069653ea2d0e90bc02ee084b08489a87dfd
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This lets you mass-validate JSON by piping instances into stdin, instead of having to put them in separate files and say
-i file.json... for every one. Each instance has to be on one line. You use it like this:You may want to do things differently, but this shows how it could be done.