Skip to content
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

Enable checking against a schema. #992

Closed
wants to merge 3 commits into from

Conversation

RichardLake
Copy link
Contributor

@RichardLake RichardLake added Enhancement Core (ckan.dll) Issues affecting the core part of CKAN Spec Issues affecting the spec Pull request labels May 31, 2015
@pjf
Copy link
Member

pjf commented Jun 21, 2015

@RichardLake : Any chance I can ask you for a rebase? I can do one myself, but our process is cleaner if we keep the same PR number. :)

@RichardLake RichardLake force-pushed the EnableSchema branch 2 times, most recently from 19a4ca8 to f5bf4f3 Compare June 21, 2015 07:29
Metapackages do not have it.
@RichardLake RichardLake force-pushed the EnableSchema branch 2 times, most recently from af49125 to f5bf4f3 Compare June 21, 2015 07:55
@Olympic1 Olympic1 added Schema Issues affecting the schema and removed Spec Issues affecting the spec labels Feb 28, 2016
@ayan4m1
Copy link
Contributor

ayan4m1 commented Aug 29, 2016

Hey, apologies for resurrecting a zombie here, but I want to move this forward in any way that would be helpful. I can try doing the... extensive rebase here, or if the code has been merged in in another form then we can maybe close this out. Does the schema checking here refer to the CKAN client itself checking the schema of the metadata files against the documented JSON schema?

@politas
Copy link
Member

politas commented Aug 29, 2016

Pretty sure this is for the Netkan/Validators to check the .ckan file being created against the schema. I'd have to do some searches for other instances to be sure, but the only other time I can see that being important is the "Install from .ckan" command, which would be the other situation we want to validate a .ckan file. Obviously, the schema included would need to be updated to the latest, and yeah, there's a lot of rebasing required.

@Olympic1 Olympic1 self-assigned this Apr 3, 2017
@Olympic1
Copy link
Member

Olympic1 commented Apr 3, 2017

Rebased in #2027

@politas politas closed this Apr 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core (ckan.dll) Issues affecting the core part of CKAN Enhancement Schema Issues affecting the schema
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants