Skip to content
This repository has been archived by the owner on Nov 28, 2019. It is now read-only.

Update ZIO to 1.0-RC1 #84

Merged
merged 3 commits into from
Mar 9, 2019
Merged

Conversation

jczuchnowski
Copy link
Contributor

@jczuchnowski jczuchnowski commented Mar 3, 2019

Could use help with rewriting AsynchronousChannel.wrap.

@ysusuk
Copy link
Contributor

ysusuk commented Mar 4, 2019

this is a lot =)

@jczuchnowski
Copy link
Contributor Author

jczuchnowski commented Mar 4, 2019

Yes, it is ;). My goal was to keep the original meaning using the new primitives.
I think I was able to do that in general, except for NetworkInterface where I noticed only SocketException is thrown.

And the missing thing is AsynchronousChannel.wrap

@jczuchnowski
Copy link
Contributor Author

jczuchnowski commented Mar 6, 2019

I went with the solution suggested by @LGLO on gitter channel.

@jczuchnowski jczuchnowski marked this pull request as ready for review March 6, 2019 23:45
} yield c1 == c2
assert(unsafeRun(testProgram))
}, {
// <<<<<<< HEAD
Copy link
Contributor

@LGLO LGLO Mar 8, 2019

Choose a reason for hiding this comment

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

cleanup pls
Why is it removed / commented out?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My bad.

@ysusuk ysusuk merged commit 6ef8739 into scalaz:master Mar 9, 2019
@jczuchnowski jczuchnowski deleted the scalaz-zio-1.0-rc1 branch April 17, 2019 17:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants