zio / interop-cats

ZIO instances for cats-effect type classes
Apache License 2.0
157 stars 67 forks source link

Cats: Stream & Sink #19

Open regiskuckaertz opened 5 years ago

regiskuckaertz commented 5 years ago

Alternative, Monad, Arrow, etc.

Where should we put them @iravid ?

iravid commented 5 years ago

Ugh. The choice is between creating interop-cats-stream and making the interop-cats depend on the stream artifact, right?

regiskuckaertz commented 5 years ago

yeah 🤕 is it safe to use optional or provided if going in the second direction in that case?

iravid commented 5 years ago

I think so. Let's ask @kaishh to make sure?

neko-kai commented 5 years ago

I think keeping one artifact is better. If the instances are in a separate object (e.g. like cats-mtl instances) then Optional is safe.