youki: Investigate to use `thiserror` in our crates
Introduction
Now we are using anyhow everywhere. But It would be better to provide some codes as crates(e.g. libcontainer, libcgroup…). Their crates in their current state are not very user-friendly of error handling.
Goal
Investigate and implement to use of thiserror instead of anyhow in crates
Tracking
Let’s do the following order because of the number of codes.
-
liboci-cliWe don’t need useanyhowin it - libcgroups https://github.com/containers/youki/pull/1872
- libcontainer
- remove
anyhow#1937
About this issue
- Original URL
- State: closed
- Created 2 years ago
- Comments: 25 (21 by maintainers)
@yihuaf and @squili , thanks for your efforts! May I ask you to add the related PR links (existing and future) in the issue description itself? I think it will help in keeping track of which all PRs were part of this change.
@yihuaf ill make sure to leave a note for the things i work on! though im going through a few things right now so it may be a bit
@yihuaf May I ask you to create TODOs list and share them with @squili ?
@squili Sorry I did not catch this thread. I made #1876 and hope I did not step on your toes.
Finished work for libcgroups, ready to review! It still uses anyhow in tests because it’s useful for tests, but it’s been removed as a dependency for everything else.
I can make a branch on a fork and PR
@squili I have given you an issue. If possible, could you please generate a plan and create a list of tasks to be done? This can be made later. Thank you.
@utam0k yeah sounds good, just assing me.