When using ironhide, I'd like it to be very clear when operations fail. Since file operations support partial failure, we currently just log the failures and process the successes and move on. In the case where all operations have failed, I think it would be better to return an error code so it's clear that it was a total failure.
This could be done by changing util::act_on_all_files to return a Result<(), String> and returning Err when there are no successes.
When using ironhide, I'd like it to be very clear when operations fail. Since file operations support partial failure, we currently just log the failures and process the successes and move on. In the case where all operations have failed, I think it would be better to return an error code so it's clear that it was a total failure.
This could be done by changing
util::act_on_all_filesto return aResult<(), String>and returningErrwhen there are no successes.