shepmaster / snafu

Easily assign underlying errors into domain-specific errors while adding context
https://docs.rs/snafu/
Apache License 2.0
1.4k stars 60 forks source link

Remove source from default `Display` implementation #356

Closed shepmaster closed 9 months ago

shepmaster commented 1 year ago

An error type with a source error should either return that error via source or include that source's error message in its own Display output, but never both.

https://github.com/rust-lang/project-error-handling/issues/27#issuecomment-763950178

So we should remove that as we do use source. Make sure to update the guide sections where we talk about the default display implementation.