-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 liftTo
to Try
, Either
and Option
#2179
Conversation
liftTo
to Try
Either
and Option
liftTo
to Try
, Either
and Option
Codecov Report
@@ Coverage Diff @@
## master #2179 +/- ##
==========================================
+ Coverage 94.71% 94.75% +0.04%
==========================================
Files 329 330 +1
Lines 5558 5568 +10
Branches 215 201 -14
==========================================
+ Hits 5264 5276 +12
+ Misses 294 292 -2
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
* res1: scala.Either[String, Int] = Left(Empty) | ||
* }}} | ||
*/ | ||
def liftTo[F[_]]: LiftToPartiallyApplied[F, A] = new LiftToPartiallyApplied(oa) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm surprised this works because LiftToPartiallyApplied
is marked private[cats]
… I guess I can have a value of this type but I can't declare or ascribe it. Interesting trick.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is the fun of working with scala.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
simple syntax extensions