The verify function currently returns Result<bool, VerifyError>, which allows ambiguous cases like Ok(false). Consider changing the return type to Result<(), VerifyError> so:
Ok(()) indicates success.
Err(VerifyError) indicates failure.
This makes the function clearer and easier to use.
The verify function currently returns
Result<bool, VerifyError>, which allows ambiguous cases likeOk(false). Consider changing the return type to Result<(), VerifyError> so:Ok(())indicates success.Err(VerifyError)indicates failure.This makes the function clearer and easier to use.