cats-effect icon indicating copy to clipboard operation
cats-effect copied to clipboard

Fold cats-retry into CE 3?

Open cb372 opened this issue 4 years ago • 7 comments

Would there be any interest in folding the features of cats-retry (i.e. a few combinators for retrying effectful actions + a DSL for building retry policies) into CE 3? Would this be a good fit for the std module?

I have to admit up front that I probably wouldn't have time to lead this, but I'd be happy to support such an effort.

cb372 avatar Nov 28 '20 14:11 cb372

I think this could be very interesting! Retry is an extremely common use-case.

djspiewak avatar Nov 28 '20 18:11 djspiewak

@cb372 Hey Chris, actually I was thinking about this and didn't even know you opened an issue about it, I was going to reach out. Would you still be ok to lend your support/approval? I can do the necessary work to integrate if so :)

SystemFw avatar Apr 07 '22 10:04 SystemFw

Hi Fabio! That would be great. I haven't touched cats-retry in months so it's slowly rotting. I'd love to see it integrated into std where it can be properly maintained.

cb372 avatar Apr 07 '22 10:04 cb372

Great, I'll open a PR then :)

SystemFw avatar Apr 12 '22 11:04 SystemFw

@SystemFw I just recently made a contribution that adds one of the examples in the docs as a convenience for a RetryPolicy that is fairly common to use. Hope this makes it either into cats-retry or the next milestone of CE. Thanks for your hard work!

nullhappens avatar Aug 30 '22 20:08 nullhappens

I guess an official update is needed on this work as well. I've struggled with motivation for months trying to get the api to be slightly smaller (to have the appropriate footprint for CE), but I think I've finally hit a turning point, hopefully this isn't far away from completion. I'll try to include the example as well :)

SystemFw avatar Aug 30 '22 20:08 SystemFw

For anyone following along here: draft PR is up and seeking feedback :)

  • https://github.com/typelevel/cats-effect/pull/3135

armanbilge avatar Sep 05 '22 00:09 armanbilge