Implement proper error types

[nbaksalyar]

Currently, all error types are returned as Box<dyn std::error::Error>, which is a quick & dirty way of doing it. We should have descriptive error types.

[blitzerr]

@nbaksalyar Do we have a preference for a crate that we want to use here ? Would Anyhow work ?

[bjorn3]

Anyhow is for executables. It doesn't allow you to match on the error. Something like thiserror makes more sense. That crate makes it easier to derive Error for regular enums.

[blitzerr]

Makes sense.

[DevinR528]

Is this issue still open I'd be interested in taking a look at adding an error type. Would you rather use thiserror or do it the long way with a bunch of Into impls?

[bjorn3]

Using thiserror is probably easier.