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

JSON Schema needs update #3590

Closed
lordsoffallen opened this issue Feb 3, 2024 · 3 comments · Fixed by #3621
Closed

JSON Schema needs update #3590

lordsoffallen opened this issue Feb 3, 2024 · 3 comments · Fixed by #3621
Labels
Community Issue/PR opened by the open-source community Issue: Bug Report 🐞 Bug that needs to be fixed

Comments

@lordsoffallen
Copy link
Contributor

Description

With the change of DataSet -> Dataset naming, currently https://raw.githubusercontent.com/kedro-org/kedro/develop/static/jsonschema/kedro-catalog-0.18.json contains the wrong configuration. This needs an update.

Context

Steps to Reproduce

Expected Result

Actual Result

-- If you received an error, place it here.
-- Separate them if you have more than one.

Your Environment

  • Kedro version used (pip show kedro or kedro -V):
  • Python version used (python -V):
  • Operating system and version:
@astrojuanlu astrojuanlu added Issue: Bug Report 🐞 Bug that needs to be fixed Community Issue/PR opened by the open-source community labels Feb 3, 2024
@astrojuanlu
Copy link
Member

Thanks @lordsoffallen! Would you like to send a PR yourself?

@lordsoffallen
Copy link
Contributor Author

hey @astrojuanlu, I would have but unfortunately i am super busy at the moment. I am also not sure if there was other breaking changes from 0.18 -> 0.19 so I suppose having a new file would be better here. If it's not solved in a month or so, I would probably have time to pick it up.

@astrojuanlu
Copy link
Member

No rush! We added this to our backlog. Anyone else feel free to pick it up in the meantime.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Community Issue/PR opened by the open-source community Issue: Bug Report 🐞 Bug that needs to be fixed
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants