You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In kanidm_client, there are many functions that return Result<bool, ClientError>, yet the bool is never used on Ok. We should change these to Result<(), ClientError> to decrease confusion. Inspired by @Firstyear's comment:
It represents past williams mistakes. I thought that there would be a case where you could set and it would be Ok() but change nothing (false), but I think we never indicate that so it should be cleaned up. That could be a new issue from this (cleanup kanidm_client bool/return values). Originally posted by @Firstyear in #465 (comment)
The text was updated successfully, but these errors were encountered:
In
kanidm_client
, there are many functions that returnResult<bool, ClientError>
, yet thebool
is never used onOk
. We should change these toResult<(), ClientError>
to decrease confusion. Inspired by @Firstyear's comment:The text was updated successfully, but these errors were encountered: