ipld / ipld/libipld

Switch to thiserror or snafu

Open
#115 12 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
150
Forks
41
PR merge metrics
No merged PRs in 30d

Description

libipld currently uses anyhow. Unfortunately:

- [According to the author](https://github.com/dtolnay/anyhow#comparison-to-thiserror), thiserror is a better fit for libraries. Anyhow makes it difficult to handle error cases.
- Anyhow doesn't have great nostd support as `?` will no longer convert errors into `anyhow::Error`.
- Anyhow requires allocation, adding an allocation to all error paths. While libipld itself requires allocation at the moment, it would be nice to eventually drop this requirement (neither the `Encode` nor `Decode` trait require allocations).

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.