-
Notifications
You must be signed in to change notification settings - Fork 523
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
Fold cats-retry into CE 3? #1459
Comments
I think this could be very interesting! Retry is an extremely common use-case. |
@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 :) |
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. |
Great, I'll open a PR then :) |
@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! |
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 :) |
For anyone following along here: draft PR is up and seeking feedback :) |
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.
The text was updated successfully, but these errors were encountered: