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

added foldF to EitherT #2780

Merged
merged 1 commit into from
Apr 9, 2019
Merged

added foldF to EitherT #2780

merged 1 commit into from
Apr 9, 2019

Conversation

kailuowang
Copy link
Contributor

@kailuowang kailuowang commented Apr 9, 2019

very simple short cuts, kind of a counterpart for getOrElseF

@kailuowang kailuowang closed this Apr 9, 2019
@kailuowang kailuowang deleted the addFoldF branch April 9, 2019 13:22
@kailuowang kailuowang restored the addFoldF branch April 9, 2019 13:27
@kailuowang
Copy link
Contributor Author

closed by mistake

@kailuowang kailuowang reopened this Apr 9, 2019
@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2780      +/-   ##
==========================================
+ Coverage   94.68%   94.68%   +<.01%     
==========================================
  Files         365      365              
  Lines        6868     6869       +1     
  Branches      293      293              
==========================================
+ Hits         6503     6504       +1     
  Misses        365      365
Impacted Files Coverage Δ
core/src/main/scala/cats/data/EitherT.scala 97.93% <100%> (+0.01%) ⬆️

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 ca425bf...629df1c. Read the comment docs.

@kailuowang kailuowang added this to the 2.0 milestone Apr 9, 2019
@kailuowang kailuowang merged commit d3b6ee7 into typelevel:master Apr 9, 2019
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