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.

Sign up for GitHub

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

Jump to bottom

Monix support #3

Open
kitlangton opened this issue Feb 7, 2022 · 4 comments
Open

Monix support #3

kitlangton opened this issue Feb 7, 2022 · 4 comments
Labels
good first issue Good for newcomers

Comments

@kitlangton
Copy link
Owner

No description provided.

@kitlangton kitlangton added the good first issue Good for newcomers label Feb 7, 2022
@kitlangton
Copy link
Owner Author

I suppose this is also supported via #7 :)

(though anyone is welcome to making a monix specific module)

@hejfelix
Copy link
Contributor

hejfelix commented Feb 8, 2022

I don't think #7 solves this in it's current state -- I made it concretely for IO and not using the type classes. I can make another PR using the type classes to allow this to be used with e.g. Monix :) but I think this should be revived.

@kitlangton kitlangton reopened this Feb 8, 2022
@kitlangton
Copy link
Owner Author

Sounds good! Tis opened :)

@hejfelix
Copy link
Contributor

I did some experimenting, and the macro fails in various ways when you are coding against the typeclass. I was unable to figure this out, unfortunately.
It seems to me, that the macro relies on having the concrete type somewhere, somehow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hejfelix @kitlangton