Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove references to Monad from CoflatMap doc #1304

Merged
merged 1 commit into from
Aug 22, 2016
Merged

Conversation

kcsongor
Copy link
Contributor

@kcsongor kcsongor commented Aug 20, 2016

Apologies in advance for this, if I had the ability to label this PR, it would definitely be something along the lines of 'nitpick'.

The doc for coflatMap states that the value is in a Monadic context, which can be confusing, because:

  • there is no Monad constraint on the context (the example being a Monad is incidental)
  • the context is more likely to be a Comonad

The doc for `coflatMap` states that the value is in a Monadic context, which can be confusing, because:
- there is no Monad constraint on the context (the example being a Monad is incidental) 
- it is *more likely* to be a Comonad
@codecov-io
Copy link

codecov-io commented Aug 20, 2016

Current coverage is 90.60% (diff: 100%)

Merging #1304 into master will not change coverage

@@             master      #1304   diff @@
==========================================
  Files           235        235          
  Lines          3607       3607          
  Methods        3549       3549          
  Messages          0          0          
  Branches         54         54          
==========================================
  Hits           3268       3268          
  Misses          339        339          
  Partials          0          0          

Sunburst

Powered by Codecov. Last update 6e12494...5fe4837

@non
Copy link
Contributor

non commented Aug 20, 2016

Makes sense to me. 👍

We'll merge this after 0.7.0 is released. Thanks!

@kcsongor
Copy link
Contributor Author

Thanks for the quick response!

@non non merged commit 79d5f53 into typelevel:master Aug 22, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants