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
Upgrade to cats-1.0.0-RC1 and more #80
Conversation
b88b884
to
32dd06b
Compare
@@ -152,7 +152,7 @@ private[effect] trait SyncInstances { | |||
|
|||
// overwriting the pre-existing one, since flatMap is guaranteed stack-safe | |||
def tailRecM[A, B](a: A)(f: A => StateT[F, S, Either[A, B]]): StateT[F, S, B] = | |||
StateT.catsDataMonadForStateT[F, S].tailRecM(a)(f) | |||
IndexedStateT.catsDataMonadForIndexedStateT[F, S].tailRecM(a)(f) |
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 have not spent any time whatsoever considering whether the instances can be generalized to IndexedStateT
. This is minimal to make it compile.
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 fine sticking with this for the time being. It seems likely that we'll be able to generalize a bit, but it'll take some looking.
Codecov Report
@@ Coverage Diff @@
## master #80 +/- ##
=======================================
Coverage 84.95% 84.95%
=======================================
Files 18 18
Lines 359 359
Branches 23 23
=======================================
Hits 305 305
Misses 54 54 |
👍, thanks! (I'll need this for catbird, which I'll need for iteratee.io, which I'll need for circe, etc.). |
Upgrades: