Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@iravid
Copy link
Member

@iravid iravid commented Mar 9, 2020

Resolves #2719
Resolves #2720
Resolves #3347
Resolves #3399
Resolves #3189

@iravid iravid force-pushed the feature/zconduit branch 2 times, most recently from 27209b9 to 98260a5 Compare March 13, 2020 07:20
@CLAassistant
Copy link

CLAassistant commented Mar 20, 2020

CLA assistant check
All committers have signed the CLA.

@iravid iravid force-pushed the feature/zconduit branch 2 times, most recently from 288b742 to 92d610f Compare April 13, 2020 20:19
@iravid
Copy link
Member Author

iravid commented Apr 13, 2020

@regiskuckaertz I force-pushed the branch to my rebase yesterday so please rebase your open branch on it 🙏🏻

@iravid iravid force-pushed the feature/zconduit branch 3 times, most recently from 374d28e to 2b8eceb Compare April 25, 2020 21:40
@iravid iravid marked this pull request as ready for review April 25, 2020 21:40
@iravid iravid requested a review from adamgfraser April 25, 2020 21:41
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 {
Copy link
Member Author

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) =>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@adamgfraser And this

@iravid iravid force-pushed the feature/zconduit branch 2 times, most recently from 9cbd419 to ba7d5c0 Compare April 27, 2020 20:16
Co-authored-by: Regis Kuckaertz <[email protected]>
Co-authored-by: Evgenii Kuznetcov <[email protected]>
@iravid iravid force-pushed the feature/zconduit branch from ba7d5c0 to a3dfc7c Compare April 27, 2020 20:21
adamgfraser
adamgfraser previously approved these changes Apr 28, 2020
Copy link
Contributor

@adamgfraser adamgfraser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@iravid iravid force-pushed the feature/zconduit branch from b3a467e to 7676377 Compare April 29, 2020 08:25
@iravid
Copy link
Member Author

iravid commented Apr 29, 2020

Will merge this after CI is green if no objections. @zio/core

@adamgfraser
Copy link
Contributor

@iravid Great!

@iravid iravid merged commit 7c564f7 into master Apr 29, 2020
michalwarecki added a commit to michalwarecki/zio that referenced this pull request Apr 30, 2020
michalwarecki added a commit to michalwarecki/zio that referenced this pull request Apr 30, 2020
iravid pushed a commit that referenced this pull request Apr 30, 2020
* ZIO Streams - improved documentation.

* Updated ZStream documentation based on #3079 change.
@mijicd mijicd deleted the feature/zconduit branch July 10, 2020 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

4 participants