-
Notifications
You must be signed in to change notification settings - Fork 520
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
Make Sync extend Defer #286
Comments
👍 can you do a PR? |
Would be cool if this made it in 1.0.0-RC3. Who's up for a PR? |
I guess I can take a look at it today @alexandru |
Merged
👍 @Avasil go for it |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
now that Defer has landed in cats 1.2.0, I think Sync should extend
Defer[A]
withdef defer[A](fa: => F[A]): F[A] = suspend(fa)
.Defer is not about side effects, but suspend can also handle those, so it seems only more general, since if side effects are allowed to happen, it can't have evaluated before returning.
The text was updated successfully, but these errors were encountered: