more error handling

[extrawurst]

I started asyncgit crate ignoring errors unwrap is all over the place but the goal would be to handle them correctly like with sync::get_commits_info

[MCord]

Hi, I would love to help but I don't have much experience in Rust. Can I take this issue ?

[extrawurst]

Hy @MCord thanks for your interest in helping❤️! Absolutely you can grab this one - let me know when you need any input!

[MCord]

So here is my first question: do we want to define a custom error for asyncgit lib or use the git2::Error ? in many cases the underlying error is git2::Error but for example in hook::hooks_commit_msg() it might be an std::io::Error or if for whatever reason we can not get a temp file name we have to return some kind of error, which type would that be ?

[extrawurst]

Good question. I was expecting to have to come up with a lib specific Error using thiserror and wrapping git2::Error,... in it to have the most flexibility. what do you think?

[MCord]

Ok, let's try that. I will send a PR so we can discuss the implementation.

[extrawurst]

Really cool! Thank you❤️!

[MCord]

I have created a pull request #58. I will continue when you give me some feedback.