ES-22: Provide OptionExt::ok_or_report to create reports with a given context for None #667

Activity

1. jnicholls commented on Jun 17, 2022

   

   jnicholls

   on Jun 17, 2022

   Author

   More actions

   Quickly created this issue on my phone, while I was thinking about it. If you accept contributions I’d be happy to contribute this, if you all are amenable to the idea.
2. TimDiekmann

   added

   area/libs > error-stackAffects the `error-stack` crate (library)Affects the `error-stack` crate (library)

   on Jun 17, 2022
3. TimDiekmann commented on Jun 17, 2022

   

   TimDiekmann

   on Jun 17, 2022

   Member

   More actions

   Hi @jnicholls! Thanks for the issue, and, yes, we'd love to see contributions!

   However, we intentionally did not provide OptionExt to convert Option::None to Report. This decision was made because an Option does not have an error to create a Report from (this is the same reason, why eyre does not provide it).
4. TimDiekmann

   closed this as not plannedon Jun 17, 2022
5. TimDiekmann

   added

   state/rejectedThis issue will not be worked on (previously "Won't Fix")This issue will not be worked on (previously "Won't Fix")

   on Jun 17, 2022
6. vilkinsons

   assigned

   TimDiekmann

   on Jun 19, 2022
7. tisonkun commented on Oct 10, 2025

   

   tisonkun

   on Oct 10, 2025

   Last edited by tisonkun

   More actions

   @TimDiekmann eyre has an OptionExt::ok_or_eyre So I'd expect something like OptionExt::ok_or_report(IntoReport)

   See fast/exn@cf0bc9d.
8. TimDiekmann commented on Oct 13, 2025

   

   TimDiekmann

   on Oct 13, 2025

   Member

   More actions

   I actually never thought about ok_or_report. A quite low-hanging fruit without unambiguity. Let's just do it!

   👍React with 👍1Reacted by tison
9. TimDiekmann

   reopened this on Oct 13, 2025
10. TimDiekmann

    changed the title [-]error-stack: Provide an OptionExt to create reports with a given context for Option::None[/-] [+]ES-22: Provide `OptionExt::ok_or_report` to create reports with a given context for `None`[/+] on Oct 13, 2025
11. TimDiekmann commented on Oct 14, 2025

    

    TimDiekmann

    on Oct 14, 2025

    Member

    More actions

    Are there any open-source volunteers available? I’m unlikely to be able to work on it in the next 1-2 weeks.
12. vilkinsons

    added

    priority/3 lowLower priority: nice-to-haveLower priority: nice-to-have

    area/libsRelates to first-party libraries/crates/packages (area)Relates to first-party libraries/crates/packages (area)

    area/dxImprovements or additions to DX or docsImprovements or additions to DX or docs

    and removed

    state/rejectedThis issue will not be worked on (previously "Won't Fix")This issue will not be worked on (previously "Won't Fix")

    on Oct 15, 2025
13. vilkinsons

    added the

    Feature

    issue type on Apr 8, 2026