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
Context: Package channel and wallet have an interface type Backend that provides a set of utility functions.
Problem: The name Backend suggests that the type provides all kinds of backend functionality while in fact it only provides a set of static utility functions. This is not evident and causes confusion when working with the code.
Suggestion: We could rename type Backend to Util or Static to clarify that it is a static utility object.
Caveat: This would affect all backend implementations. However, if we want to change it, it's better to do it sooner than later.
The text was updated successfully, but these errors were encountered:
Since we thought about removing DecodeSig, we could then rename the wallet Backend.
We could also split it in two interfaces if that makes it clearer, the SetBackend function would then accept multiple arguments.
Context: Package channel and wallet have an interface type
Backend
that provides a set of utility functions.Problem: The name
Backend
suggests that the type provides all kinds of backend functionality while in fact it only provides a set of static utility functions. This is not evident and causes confusion when working with the code.Suggestion: We could rename type
Backend
toUtil
orStatic
to clarify that it is a static utility object.Caveat: This would affect all backend implementations. However, if we want to change it, it's better to do it sooner than later.
The text was updated successfully, but these errors were encountered: