Skip to content

jsonschema: generate from schema #4161

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

Closed
wants to merge 2 commits into from

Conversation

skshetry
Copy link
Collaborator

@skshetry skshetry commented Jul 3, 2020

  • ❗ I have followed the Contributing to DVC checklist.

  • πŸ“– If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here.

  • ❌ I will check DeepSource, CodeClimate, and other sanity checks below. (We consider them recommendatory and don't expect everything to be addressed. Please fix things that actually improve code or fix bugs.)

Thank you for the contribution - we'll try to review it as soon as possible. πŸ™

@skshetry skshetry self-assigned this Jul 3, 2020
@@ -0,0 +1,258 @@
{
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@@ -0,0 +1,109 @@
try:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Right now, it does not depend on any of the DVC. The schema is just independent duplicated. But, it cannot be used for validation, at least not yet. πŸ™

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Same here, if we don't want to maintain it, could push to separate repo.

@skshetry skshetry marked this pull request as draft July 3, 2020 13:13
@skshetry
Copy link
Collaborator Author

skshetry commented Jul 3, 2020

Will do it in a separate repo.

@skshetry skshetry closed this Jul 3, 2020
@skshetry skshetry deleted the gen-jsonschema branch July 3, 2020 13:32
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.

1 participant