Skip to content

Add normative references for 3 json schema versions #170

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

Merged
merged 3 commits into from
Jul 6, 2023
Merged

Conversation

decentralgabe
Copy link
Collaborator

@decentralgabe decentralgabe commented Jul 5, 2023

fix #133


Preview | Diff

@decentralgabe
Copy link
Collaborator Author

Screenshot 2023-07-05 at 1 09 58 PM

index.html Outdated
• <a href="https://datatracker.ietf.org/doc/html/draft-bhutton-json-schema-00">JSON Schema: A Media Type for Describing JSON Documents</a></br>
• <a href="https://datatracker.ietf.org/doc/html/draft-bhutton-json-schema-validation-00">JSON Schema Validation: A Vocabulary for Structural Validation of JSON</a></br>
• <a href="https://datatracker.ietf.org/doc/html/draft-bhutton-relative-json-pointer-00">Relative JSON Pointers</a>
• <a href="https://datatracker.ietf.org/doc/html/draft-handrews-json-schema-02">JSON Schema: A Media Type for Describing JSON Documents</a></br>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure about all these IETF refs, prefer to not mix IETF and OpenJS refs for the same standard, I suggest not referring to IETF drafts at all.

Copy link
Collaborator

@OR13 OR13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove the IETF refs, just use OpenJS

@decentralgabe
Copy link
Collaborator Author

updated
Screenshot 2023-07-05 at 1 25 25 PM

@OR13
Copy link
Collaborator

OR13 commented Jul 5, 2023

Given that we will be forced to review this all when we advance to CR, any reason not to merge this now?

@iherman
Copy link
Member

iherman commented Jul 6, 2023

Given that we will be forced to review this all when we advance to CR, any reason not to merge this now?

I agree.

Actually: this blog on the stability of JSON Schemas is interesting (and important). As we get closer to CR (or even PR) we may have a stable reference that we can rely on once and for all. (I missed this particular blog, my attention was just drawn to it right now.)

In the meantime, we can simply go on with this PR.

@decentralgabe decentralgabe merged commit 6cfae46 into main Jul 6, 2023
@decentralgabe decentralgabe deleted the schema-refs branch July 6, 2023 17:02
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.

Being able to normatively reference JSON Schema
3 participants