-
Notifications
You must be signed in to change notification settings - Fork 123
GEN-150: ResultExt doesn't cover Report<EyreReport> for some reason #4295
Copy link
Copy link
Open
Labels
area/libsRelates to first-party libraries/crates/packages (area)Relates to first-party libraries/crates/packages (area)area/libs > error-stackAffects the `error-stack` crate (library)Affects the `error-stack` crate (library)category/bugSomething isn't workingSomething isn't workinglang/rustPull requests that update Rust codePull requests that update Rust codepriority/2 mediumMedium priority: needs to be doneMedium priority: needs to be done
Description
Activity
Metadata
Metadata
Assignees
Labels
area/libsRelates to first-party libraries/crates/packages (area)Relates to first-party libraries/crates/packages (area)area/libs > error-stackAffects the `error-stack` crate (library)Affects the `error-stack` crate (library)category/bugSomething isn't workingSomething isn't workinglang/rustPull requests that update Rust codePull requests that update Rust codepriority/2 mediumMedium priority: needs to be doneMedium priority: needs to be done
Describe the bug
For a reason I haven't yet understood,
ResultExttrait doesn't work onerror_stack::Result(e.g.change_contextis not found), althoughEyreReportseems to satisfy theContexttrait.However, it is possible to call
Report::change_contexton it without a problem.To reproduce
https://github.com/0x009922/error-stack-eyre-compat-issue
Expected behavior
Expected
.into_report().change_context(...)to workRust compiler
1.76.0 (07dca489a 2024-02-04)
Host
aarch64-apple-darwin
Target
aarch64-apple-darwin
Version
0.4.1
Features
default, eyre
Additional context
Compiler error: