Open sullivan- opened 7 years ago
I'd like to mention that I myself am only using this adapter in a test suite, so getting a better implementation here may well be a lot more important to you than it is to me. Of course, I want to see this project beautiful and complete. But I have a lot of priorities to balance. If you care, please consider chipping in! Thank you.
Current implementation limitations:
Bimonad
instead of just aMonad
. I'm not sure if this is negotiable or not. It seems like we need to callF.extract
on the iteratee to get it to "run".Future
from the global execution context to kick off the producer asynchronously. A more catsy approach, as described in the last bullet point, should do away with this. I didn't want to add an implicitExecutionContext
argument here, as that seems like an implementation detail, and a rewrite of this method without it would not be backwards compatible. Maybe there is some way to pull this off withTask
? I tried, couldn't figure it out