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

collect all errors before exiting validation #11

Open
matthewcornell opened this issue Sep 24, 2024 · 0 comments
Open

collect all errors before exiting validation #11

matthewcornell opened this issue Sep 24, 2024 · 0 comments

Comments

@matthewcornell
Copy link
Collaborator

matthewcornell commented Sep 24, 2024

per @bsweger comment:

The the change I do think we should make is bunching together error messages in the validations, so people don't have to make one change at a time to get this working (i.e., they get a list of everything that didn't validate so they can fix it all up)

And this one:

If there are multiple validation errors (e.g., missing quantile levels and not a designated model), it would be user-friendly to return all of them at once (instead of raising an error on the first failure).

Originally posted by @bsweger in #10 (review)

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

1 participant