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
The variant operator has a strange limitation in the form of the need to cast the store to a string. It seems to me that its api needs to be reworked in order to make the operator more flexible.
I've left the pr with the implementation of this operator (cases) separate from the variant if that idea sounds good.
P.S. I'm not sure, maybe my idea is bad, I'll be grateful if you help me understand this. And please forgive me for my bad english.
The text was updated successfully, but these errors were encountered:
But I don't really like the idea of making it a new operator, since it more like an extended version of current variant - it would be confusing to have a few operators for the same purpose 🤔
I think, it could very well be an extension of variant, what do you think?
This way current variant will be basically a shorthand of your, more flexible, version
As i see it based on your PR - it could be merged with variant easily 🤔
The
variant
operator has a strange limitation in the form of the need to cast the store to a string. It seems to me that its api needs to be reworked in order to make the operator more flexible.Some example of using:
I've left the pr with the implementation of this operator (
cases
) separate from thevariant
if that idea sounds good.P.S. I'm not sure, maybe my idea is bad, I'll be grateful if you help me understand this. And please forgive me for my bad english.
The text was updated successfully, but these errors were encountered: