mimblewimble / grin

Minimal implementation of the Mimblewimble protocol.
https://grin.mw/
Apache License 2.0
5.04k stars 990 forks source link

Tracking issue for deprecation of `failure` crate #3631

Closed trevyn closed 3 years ago

trevyn commented 3 years ago

Per its README, the failure crate is deprecated:

Notice: failure is deprecated. If you liked failure's API, consider using:

  • Anyhow is a good replacement for failure::Error.
  • thiserror is a good, near drop-in replacement for #[derive(Fail)].

I agree with the suggestion to migrate to anyhow and thiserror, and will investigate it unless anyone has other suggestions or objections.

Investigating on branch https://github.com/trevyn/grin/tree/trevyn-deprecate-failure. To do:

antiochp commented 3 years ago

I'm :+1: on exploring replacements for failure here. We have had multiple iterations of our error handling, none of which have felt 100% right. And we are definitely not the only rust project to struggle with this.

trevyn commented 3 years ago

PR in progress: #3633