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

Add BiProd, fixes #874 #906

Closed
wants to merge 1 commit into from
Closed

Conversation

adelbertc
Copy link
Contributor

No description provided.

@codecov-io
Copy link

Current coverage is 89.45%

Merging #906 into master will decrease coverage by -0.14% as of 8d08d62

@@            master    #906   diff @@
======================================
  Files          181     182     +1
  Stmts         2423    2447    +24
  Branches        76      76       
  Methods          0       0       
======================================
+ Hit           2171    2189    +18
  Partial          0       0       
- Missed         252     258     +6

Review entire Coverage Diff as of 8d08d62

Powered by Codecov. Updated on successful CI builds.

@ceedubs
Copy link
Contributor

ceedubs commented Mar 8, 2016

@adelbertc could you provide some example use-cases? If we decide to go forward with this, I'd like to see some more documentation.

@adelbertc
Copy link
Contributor Author

@ceedubs I confess I haven't had a use for this myself. I've definitely had uses for Prod, especially in the context of evaluating algebras. The motivation behind this was primarily for consistency in the sense that there is product composition for Functor, but none for Bifunctor.

@adelbertc adelbertc mentioned this pull request Apr 4, 2016
@adelbertc adelbertc closed this Jul 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants