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

Migration from failure #100

Closed
l4l opened this issue Jun 9, 2020 · 1 comment
Closed

Migration from failure #100

l4l opened this issue Jun 9, 2020 · 1 comment

Comments

@l4l
Copy link
Contributor

l4l commented Jun 9, 2020

failure is deprecated so I would suggest to replace it with either thiserror or snafu, both of them are compatible with std::error::Error. I may help you with a migration itself, but need to decide which one to choose. Personally I like the first one since it smaller and simpler to use, but the former allows more explicit error handling.
suharev7 added a commit that referenced this issue Jun 9, 2020
@suharev7
Switch to thiserror for error handling #100
05b0ad1
suharev7 added a commit that referenced this issue Jun 9, 2020
@suharev7
Switch to thiserror for error handling #100
5c8c79f
@suharev7
Copy link
Owner

suharev7 commented Jun 9, 2020

Thanks for your suggestion. I've migrated into thiserror in v0.1.21 and v0.2.0-alpha.5.

@suharev7 suharev7 closed this as completed Jun 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@suharev7 @l4l