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

Use submodules #145

Merged
merged 3 commits into from
Sep 19, 2017
Merged

Use submodules #145

merged 3 commits into from
Sep 19, 2017

Conversation

adamvoss
Copy link
Contributor

This utilizes Git submodules to the spec repository for specification artifacts. This both ensures the served version matches the released version and shows where in history the artifacts come from.

Note: The desired hash for draft-05 and draft-06 could change as a result of json-schema-org/json-schema-spec#368.

Copy link
Contributor

@handrews handrews left a comment

Choose a reason for hiding this comment

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

I like this idea. I would like to get at least one additional approval from either @Relequestual or @awwright as I have not always correctly guessed how other folks prefer to manage the connections between the repos.

But I think this is a significant improvement.

@Relequestual
Copy link
Member

Just one query. Where does "current" go / end up?

@adamvoss
Copy link
Contributor Author

adamvoss commented Sep 1, 2017

Just one query. Where does "current" go / end up?

Good catch: it ends up absolutely nowhere! Since it was the first entry clearly it was something I added while I was experimenting and I must not have gotten it completely removed. I have pushed a commit to do that now.

@handrews
Copy link
Contributor

Looks like all concerns were met, so I'm going to go ahead and merge this.

@handrews handrews merged commit b1435c5 into json-schema-org:master Sep 19, 2017
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