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

Improve error handling story #7

Open
begleynk opened this issue Jan 14, 2024 · 0 comments
Open

Improve error handling story #7

begleynk opened this issue Jan 14, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@begleynk
Copy link
Member

begleynk commented Jan 14, 2024

So far Dossier has taken a "panic when you see something weird" attitude. This has been great getting the project off the ground quickly, but it won't work going forward.

I think the philosophy should instead be "we'll do our best to parse your input, even if it is malformed, and try to return something". We can output warnings into STDERR if there is a weird output, but generally we shouldn't panic like we do currently if we see something unexpected.

We could also have an option to make turn warnings into errors and return a non-zero status if the user so chooses.

@begleynk begleynk added the enhancement New feature or request label Jan 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant