IceDynamix / reliquary-archiver

tool to create a relic export from network packets of a certain turn-based anime game
MIT License
184 stars 18 forks source link

Use `anyhow` for prettier errors on the console #46

Open FichteFoll opened 2 months ago

FichteFoll commented 2 months ago

Since this application does not have a GUI at the moment but has a comparatively wide target audience, some clearer error message such as those provided by anyhow instead of the default output of panics on Result::unwrap (like in #33) might be helpful for users to figure out their issues more quickly (on their own).

Just an idea I'm putting up here.

Also, implementing proper bubbling of Errs until the main method will also be beneficial when a GUI is eventually added.

IceDynamix commented 2 months ago

related https://github.com/IceDynamix/reliquary/pull/3

FichteFoll commented 2 months ago

Ah, I didn't even notice there was a separate library project. Indeed, reliquary should be using thiserror, whereas anyhow makes more sense for reliquary-archiver as the user-facing component.