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 Commutative{Monad, FlatMap} instances for IdT #2104

Merged
merged 1 commit into from
Dec 14, 2017

Conversation

ceedubs
Copy link
Contributor

@ceedubs ceedubs commented Dec 14, 2017

No description provided.

@codecov-io
Copy link

codecov-io commented Dec 14, 2017

Codecov Report

Merging #2104 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2104      +/-   ##
==========================================
+ Coverage   94.79%   94.79%   +<.01%     
==========================================
  Files         325      325              
  Lines        5490     5492       +2     
  Branches      214      221       +7     
==========================================
+ Hits         5204     5206       +2     
  Misses        286      286
Impacted Files Coverage Δ
core/src/main/scala/cats/data/IdT.scala 97.72% <100%> (+0.1%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 51635f8...4bbceb1. Read the comment docs.

Copy link
Member

@LukaJCB LukaJCB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks! :)

@kailuowang kailuowang merged commit d9c3c5b into typelevel:master Dec 14, 2017
@ceedubs ceedubs deleted the idt-commutative-monad branch December 14, 2017 14:54
@kailuowang kailuowang added this to the 1.0.0 milestone Dec 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants