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

Adopt next's parameter validation #359

Open
adswa opened this issue Mar 1, 2023 · 2 comments
Open

Adopt next's parameter validation #359

adswa opened this issue Mar 1, 2023 · 2 comments

Comments

@adswa
Copy link
Member

adswa commented Mar 1, 2023

MetaLad commands define parameter constraints in the same way that datalad core does. The newly introduced parameter validation in datalad-next provides a more powerful re-implementation of this, though. #357 (comment) sketches what would be needed for a single command.

The hackathon and a call earlier today brought a few issues to light where parameter validation could have caught errors or problems early on, so I think metalad would benefit a lot from adopting it. However, it would bring a dependency to datalad-next, which would be new. What do you think, @christian-monch and @mih?

@christian-monch
Copy link
Collaborator

Good idea, let's go for it.

@adswa
Copy link
Member Author

adswa commented Mar 2, 2023

I have made a small demo in #362 for meta_extract

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

No branches or pull requests

2 participants