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

Improved error handling #83

Open
0Mayank opened this issue Oct 22, 2024 · 2 comments
Open

Improved error handling #83

0Mayank opened this issue Oct 22, 2024 · 2 comments

Comments

@0Mayank
Copy link

0Mayank commented Oct 22, 2024

Hi,

We use ripunzip as a crate in our app, and wanted to suggest propagating ripunzip errors using an enum instead of anyhow, it would allow us to handle them and show appropriate error messages to the user.

@adetaylor
Copy link
Collaborator

Thanks - yep, I'd be happy to accept a pull request to switch over to an enum (possibly using thiserror).

@0Mayank
Copy link
Author

0Mayank commented Oct 22, 2024

Great, I'm a bit busy this week, but I'll work on it and submit a pr soon.

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