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 XorT#fromEither #1023

Merged
merged 1 commit into from
May 10, 2016
Merged

Add XorT#fromEither #1023

merged 1 commit into from
May 10, 2016

Conversation

markus1189
Copy link
Contributor

@markus1189 markus1189 commented May 10, 2016

Adds the ability to call XorT.fromEither to lift from Either[A,B] . This is sugar for the more verbose x => XorT(Xor.fromEither(x)) and complements the ability to go from XorT to Either via toEither

@codecov-io
Copy link

codecov-io commented May 10, 2016

Current coverage is 88.38%

Merging #1023 into master will increase coverage by +<.01%

  1. 3 files (not in diff) in .../main/scala/cats/std were modified. more
  2. 2 files (not in diff) in ...n/scala/cats/functor were modified. more
@@             master      #1023   diff @@
==========================================
  Files           215        215          
  Lines          2702       2703     +1   
  Methods        2633       2638     +5   
  Messages          0          0          
  Branches         64         60     -4   
==========================================
+ Hits           2388       2389     +1   
  Misses          314        314          
  Partials          0          0          

Powered by Codecov. Last updated by 4285a07...4017e72

@markus1189 markus1189 changed the title Add XorT#apply for Either Add XorT#fromEither May 10, 2016
@adelbertc
Copy link
Contributor

👍 on green

@ceedubs
Copy link
Contributor

ceedubs commented May 10, 2016

👍 thanks!

@ceedubs ceedubs merged commit db1fc3e into typelevel:master May 10, 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.

4 participants