Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Use registry in datapackage-validate #29

Closed
pwalsh opened this issue May 26, 2015 · 3 comments
Closed

Use registry in datapackage-validate #29

pwalsh opened this issue May 26, 2015 · 3 comments

Comments

@pwalsh
Copy link
Member

pwalsh commented May 26, 2015

Currently is hardcoded for the default Data Package schema.

Need to:

  • Allow passing in any (valid) data package schema
  • Support datapackage-registry directly?
@pwalsh pwalsh added this to the Backlog milestone May 26, 2015
@gvidon
Copy link
Contributor

gvidon commented May 27, 2015

@pwalsh currently datapackage-validate takes just data package json as a param and validate it over limited set of predefined schemas. Perhaps would be good to have second param for validation — schema . This way datapackage-validate will never deal with a schema but get everything from params. Not sure if it brake any design concepts.

@pwalsh
Copy link
Member Author

pwalsh commented May 27, 2015

Yes, I think we will do something like that. The benefit of us publishing datapackage-registry on NPM, and then using that to get a schema on datapackage-validate, is that we ensure our "official" packages are always available to the validator. But I do think we should also change datapackage-validate to accept any schema that gets passed to it from the calling code.

In any event, we'll get back to this a bit later - perhaps in the next milestone.

@pwalsh
Copy link
Member Author

pwalsh commented Jun 4, 2015

This is is replaced by #27 and openknowledge-archive/datapackage-validate-js#6

@pwalsh pwalsh closed this as completed Jun 4, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants