-
Couldn't load subscription status.
- Fork 1.4k
ZConduit #3079
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
ZConduit #3079
Conversation
27209b9 to
98260a5
Compare
288b742 to
92d610f
Compare
|
@regiskuckaertz I force-pushed the branch to my rebase yesterday so please rebase your open branch on it 🙏🏻 |
374d28e to
2b8eceb
Compare
| case (Some(Right(l)), Some(Left(r))) => Some(l.zipWith(r)(f)) | ||
| case (Some(Left(l)), Some(Right(r))) => Some(l.zipWith(r)(f)) | ||
| case _ => None | ||
| left.zipAllWith(right)(l => (Some(l), None), r => (None, Some(r)))((l, r) => (Some(l), Some(r))).collectWhile { |
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.
@adamgfraser Please double-check this
| val value = f(self.value, that.value) | ||
| val shrink = self.shrink | ||
| .combine[R1, Nothing, State, Sample[R1, B], Sample[R1, C]](that.shrink)((false, false, None, None)) { | ||
| case ((leftDone, rightDone, s1, s2), left, right) => |
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.
@adamgfraser And this
9cbd419 to
ba7d5c0
Compare
Co-authored-by: Regis Kuckaertz <[email protected]> Co-authored-by: Evgenii Kuznetcov <[email protected]>
ba7d5c0 to
a3dfc7c
Compare
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.
Looks good.
b3a467e to
7676377
Compare
|
Will merge this after CI is green if no objections. @zio/core |
|
@iravid Great! |
* ZIO Streams - improved documentation. * Updated ZStream documentation based on #3079 change.
Resolves #2719
Resolves #2720
Resolves #3347
Resolves #3399
Resolves #3189