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.

Sign up for GitHub

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

Jump to bottom

Move kedro-catalog JSON schema to kedro-datasets #4258

Open
ianwhale opened this issue Oct 25, 2024 · 2 comments
Open

Move kedro-catalog JSON schema to kedro-datasets #4258

ianwhale opened this issue Oct 25, 2024 · 2 comments
Labels
good first issue Good first issues for beginners Issue: Feature Request New feature or improvement to existing feature

Comments

@ianwhale
Copy link
Contributor

Description

Summary here is:

  • Datasets have moved, so it seems like this file should too.
  • Workflow for adding new datasets will be more seamless with the file living with kedro-datasets.

See more on the origin of this issue here: #4196.

Context

Alleviate maintenance burden by having these schema live with the datasets themselves.

Possible Implementation

  • Move it on over!
  • Add clause on "update json schema if necessary" to PR template.

Possible Alternatives

None considered!
@ianwhale ianwhale added the Issue: Feature Request New feature or improvement to existing feature label Oct 25, 2024
@datajoely
Copy link
Contributor

I would add that it might be time to explore automatic generation through introspection and avoid the manual maintenance element

@ianwhale
Copy link
Contributor Author

If only Hacktober had more time! That would be a fun little project.

Great idea Joel!

@noklam noklam added Hacktoberfest Issue suitable for hacktoberfest good first issue Good first issues for beginners labels Oct 27, 2024
@github-actions github-actions bot mentioned this issue Nov 1, 2024
Open
@noklam noklam mentioned this issue Nov 4, 2024
Open
@merelcht merelcht removed the Hacktoberfest Issue suitable for hacktoberfest label Nov 26, 2024
@merelcht merelcht moved this to To Do in Kedro Framework Nov 26, 2024
chrisschopp added a commit to chrisschopp/kedro that referenced this issue Dec 1, 2024
@chrisschopp
Move kedro-catalog JSON schema to kedro-datasets kedro-org#4258
0a37127 
Signed-off-by: Chris Schopp <56572144+chrisschopp@users.noreply.github.com>
This was referenced Dec 1, 2024
Open
Merged
@chrisschopp chrisschopp mentioned this issue Dec 12, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good first issues for beginners Issue: Feature Request New feature or improvement to existing feature
Projects
Kedro Framework
Status: To Do
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ianwhale @noklam @datajoely @merelcht