-
Notifications
You must be signed in to change notification settings - Fork 17
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
Provide cats-effect integration #2
Comments
@alexandru thank you for creating this issue! Yes, it sounds reasonable to provide the type classes in sub-modules. I'll work on it after the initial release. |
Hi @fwbrasil, are you still working on this? I'd be happy to take this ticket if not :) |
Hey @LukaJCB, I’m not working on it. It’d be great if you could give it a try! |
I opened a PR :) |
@LukaJCB thanks for working on this, it’s great for validation of the upcoming Cats-Effect 1.0.0 👍 |
I know of how you feel of having Cats as dependency 🙂 but providing cats-effect integration in a different sub-project would be cool.
Consider that cool libraries like Monix, FS2, Http4s, Doobie and others now have
cats-effect
integration and would be usable with yourTask
implementation, if those type class instances are provided.You also get tests for free via
cats-effect-laws
, so it would be a win at least for that 🙂The text was updated successfully, but these errors were encountered: