search

bb30 / adbackup

backup tool for android written in rust
MIT License
4 stars 2 forks source link

better error handling in devices.rs #8

Closed DonatJR closed 6 years ago

DonatJR commented 6 years ago

use failure for error handling in devices.rs as indicated by the FIXME's in the code

we could do this together for the first time if you want to, so we can both get acquainted with the failure-module

SailReal commented 6 years ago

learn-duel took too much time last week so there was no time left to dive into this issue together.

Just mentioned #4 in the commit but it is also relevant for this issue here: https://github.com/DonatJR/adbackup/commit/c30191a6fe4b9be9eeda1131476c6497877d665e What do you think about that strategy?

DonatJR commented 6 years ago

looks good to me and you are more familiar with the error_chain concepts, so 👍