-
-
Notifications
You must be signed in to change notification settings - Fork 279
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add a relative-json-pointer format #417
Conversation
We have generally included formats necessary for describing JSON Schema vocabularies, and JSON Hyper-Schema now uses Relative JSON Pointers.
This is more clear than the regular expression, and more accurate.
We might have to pull in the text of "draft-luff-relative-json-pointer", or figure out how to advance that draft. And in my opinion, I'm still not really sure if relative pointers are actually ever necessary. |
How would you handle #385 and #386 without them? And without making up some other arbitrary new format, because there is no reason to introduce an entirely different syntax when there is one that is compatible with the syntax (JSON Pointer) that we already use extensively. |
@awwright I emailed Geraint and he's fine with me/us re-submitting the I-D or pulling it in. His thoughts:
I'm inclined to just rebuild the XML (Geraint could not find the sources) and re-submit it as a separate I-D myself leaving Geraint as the author and putting myself as editor on the grounds of doing the mechanics to get it going again (he and I discussed this in email). It hardly matters at this stage, and that's just a lot easier than figuring out the best way to merge it into JSON Schema. |
@awwright and I chatted off-github and (I think) agreed that |
@awwright this is coming up on 2 weeks in a couple of days. Given our discussion offlist I'm going to assume you've dropped your objections to this unless you comment otherwise. |
We have generally included formats necessary for describing
JSON Schema vocabularies, and JSON Hyper-Schema now uses
Relative JSON Pointers.
Also use it in the links meta-schema (as a separate commit).
I did not bother to file an issue for this as it seems pretty straightforward
and in line with existing precedent. Also, it was faster to write a PR
so even if rejected this was less work :-)