Skip to content
This repository was archived by the owner on Nov 2, 2023. It is now read-only.

Update the web site with the new draft. #42

Closed
wants to merge 2 commits into from

Conversation

handrews
Copy link
Contributor

@handrews handrews commented Nov 3, 2016

Tried to acknowledge that this is the fifth draft while preserving the proper IETF draft author+numbering scheme. Also needed to change the URIs compared to what's in json-schema-spec as they were never changed from "04" before things were tagged. Otherwise these are copied directly over.

These were copied from the draft-wright-json-schema-00 tag in the
json-schema-org/json-schema-spec repository.

Addresses issue json-schema-org#38.

Also remove "links" as it was part of older drafts and is no longer
a separate meta-schema.
@Relequestual
Copy link
Member

Fixes #38

@Relequestual
Copy link
Member

Also needed to change the URIs compared to what's in json-schema-spec as they were never changed from "04" before things were tagged.

@awwright Should this result in a minor point release?

@Relequestual
Copy link
Member

If there are no objections or queries over the weekend, I'll merge early next week (although may need a prompt!)

@awwright
Copy link
Member

awwright commented Nov 4, 2016

The meta-schema did not change with the new draft, the URI is the same
(including -04). There are no new meta-schemas. Idk if this was made
sufficiently clear, the I-D should list the URIs to use for meta-schemas. I
can't seem to find it... Which would be a problem.

We should be updating the website's links to the latest draft though, but I
have been away from my desktop all week... Feel free to update those.

I've been working on a use cases document, I'll publish that sometime too.

On Nov 4, 2016 04:47, "Ben Hutton" notifications@github.com wrote:

If there are no objections or queries over the weekend, I'll merge early
next week (although may need a prompt!)


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#42 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAatDaT6H0HWHsvsjRZhv28H0NcgqZyBks5q6xtkgaJpZM4Ko5_5
.

@Relequestual
Copy link
Member

@awwright Confused. Metaschema wasn't mentioned. Specifically, a commit on this PR: b458a11

@handrews
Copy link
Contributor Author

handrews commented Nov 4, 2016

Actually, now I'm even more confused. @awwright The meta-schemas should have been changed, particularly the meta-hyper-schema. You took out fragmentResolution, added base, and changed methods in such a way that the description in the meta-hyper-schema is now invalid. Plus, in the regular meta-schema, we fixed the bug that format had been omitted, and several uses of format should have been changed from uri to uriref.

Should we perhaps rush out another draft so that we can have usable meta-schemas? The ones at the tagged point in the spec repo do not match the current draft at all.

@handrews
Copy link
Contributor Author

handrews commented Nov 5, 2016

@awwright has made it clear that this isn't going to work. I'll re-submit a new pull request with the meta-schema update removed.

@handrews handrews closed this Nov 5, 2016
@handrews handrews deleted the draft-wright-00 branch April 2, 2017 23:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants