Skip to content
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

Open
alexandru opened this issue Apr 16, 2018 · 5 comments
Open

Provide cats-effect integration #2

alexandru opened this issue Apr 16, 2018 · 5 comments

Comments

@alexandru
Copy link

alexandru commented Apr 16, 2018

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 your Task 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 🙂

@fwbrasil
Copy link
Collaborator

@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.

@LukaJCB
Copy link

LukaJCB commented Aug 24, 2018

Hi @fwbrasil, are you still working on this? I'd be happy to take this ticket if not :)

@fwbrasil
Copy link
Collaborator

Hey @LukaJCB, I’m not working on it. It’d be great if you could give it a try!

@LukaJCB
Copy link

LukaJCB commented Aug 24, 2018

I opened a PR :)

@alexandru
Copy link
Author

@LukaJCB thanks for working on this, it’s great for validation of the upcoming Cats-Effect 1.0.0 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants