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
decouple from frees-async #297
Conversation
bring the listenableFuture2Async function to the project, and don't use frees-async-* packages
@@ -79,4 +80,20 @@ object monixCalls { | |||
outputObserver | |||
)) | |||
) | |||
|
|||
private[this] def listenableFuture2Async[F[_], A]( | |||
fa: => ListenableFuture[A])(implicit F: Async[F], E: ExecutionContext): F[A] = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've changed teh AsyncContext[F]
requirement with an Async[F]
requirement, which is the same idea.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we can restrict the implicit req to the Effect
typeclass?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Effect
is an Async
, so it would be widening. Also, we're not using runAsync
on the F
, so we don't need Effect
, right?
is this fixing #297 ? |
Ah yes, I forgot mentioning it in the comment |
I'd suggest removing this: https://github.com/frees-io/freestyle-rpc/blob/master/project/ProjectPlugin.scala#L54. We can use the grpc guava version, which it's already available in the classpath. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bring the listenableFuture2Async function to the project, and don't
use frees-async-* packages
FIXES #297