You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
But it doesn't, because the Parallel instance for Either is in cats.instances.parallel, not cats.instances.either
After #3033, the only instances in the parallel instance package are for Either, Stream, and the NonEmptyParallel instances for List and Vector. I think all of these should be moved the instance packages for those types—cats.instances.either, cats.instances.stream.
This change would break source compatibility, but could be done in a binary-compatible way in 2.1 (by making all of the contents of cats.instance.parallel non-implicit and deprecated).
The text was updated successfully, but these errors were encountered:
@barambani I've not done anything on it, so that sounds good to me! I think it's a pretty reasonable change that could happen in 2.1, but we'll want to make sure there's consensus about that.
It seems pretty reasonable to expect the following to work:
But it doesn't, because the
Parallel
instance forEither
is incats.instances.parallel
, notcats.instances.either
After #3033, the only instances in the
parallel
instance package are forEither
,Stream
, and theNonEmptyParallel
instances forList
andVector
. I think all of these should be moved the instance packages for those types—cats.instances.either
,cats.instances.stream
.This change would break source compatibility, but could be done in a binary-compatible way in 2.1 (by making all of the contents of
cats.instance.parallel
non-implicit and deprecated).The text was updated successfully, but these errors were encountered: