-
Notifications
You must be signed in to change notification settings - Fork 376
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
should Chain properties depends on fantasy-land/ap (and possibly .map)? #334
Comments
In my personal (contested) view it does, but not as explicitly. The dependency is captured in the Derivations section of the spec, where a relationship between
I read this as: "A Chain must implement EDIT: Another way to think of it is that the Apply interface is like a weaker Chain interface: Apply can be defined in terms of Chain, but not the other way around. Since Apply is a weak Chain, Chain is an extended Apply. So we have the arrow |
Well, the derivation section uses the "may" expression, not the "must". IMHO, an explicit property should be added to the |
I agree with you. I think the "may"-expression in the derivations section is a source of confusion. It's using "may" to suggest that a developer doesn't have to derive their implementation of The reason they "may" choose to do so, is because it is assumed (but never explicitly stated) that this is always possible. I think the spec should be updated to make this more clear. Furthermore, it is never made clear whether the non-derived implementation of I've seen this discussion play out in other threads on this repo as well. A quick search came up with #214 but I think there might be more. |
I see. If I look through all the definitions I find other examples (traversable, for instance). Anyway, I'll follow your suggestion: I'll consider the derivation section mandatory. Thanks! |
It's more like a question than an actual issue:
Since Chain must implement the Apply specification, I'm expecting that one of the Chain properties should depends on something related to the Apply properties (as, for instance, Applicative's
fantasy-land/of
properties resort to thefantasy-land/ap
method).The text was updated successfully, but these errors were encountered: