Skip to content
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

Reconsider name of zipsWith' #15

Open
treeowl opened this issue Sep 13, 2017 · 6 comments
Open

Reconsider name of zipsWith' #15

treeowl opened this issue Sep 13, 2017 · 6 comments

Comments

@treeowl
Copy link
Contributor

treeowl commented Sep 13, 2017

I was hoping someone would come up with a better name before that PR got merged.

@andrewthad
Copy link
Contributor

Ideas:

  • zipsLayerWith
  • zipsLayer
  • zipsFunctor

I'm not crazy about any of them though. Also, I just remembered that the haddocks for that function will need to be adjusted to put the MonadPlus laws in a code block.

@treeowl
Copy link
Contributor Author

treeowl commented Sep 13, 2017 via email

@andrewthad
Copy link
Contributor

As a look at it more, I realize that I don't understand what is meant by "satisfying the MonadPlus laws" in this context. After all, zipsWith' is just a function. Why does it need to satisfy any laws. Isn't the onus for that usually on the definitions of mzero and mplus, not on other functions?

@treeowl
Copy link
Contributor Author

treeowl commented Sep 13, 2017 via email

@treeowl
Copy link
Contributor Author

treeowl commented Sep 14, 2017 via email

@treeowl
Copy link
Contributor Author

treeowl commented Sep 15, 2017

We should also reconsider the names of all the functions currently flagged Post. Or maybe even stick them all in a separate module. Let's try to do that pretty soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants