Skip to content
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

Relax some typeclass constraints #1587

Merged
merged 2 commits into from Dec 11, 2017
Merged

Conversation

@aeons
Copy link
Member

@aeons aeons commented Dec 8, 2017

Now that the different run methods on Streams just require a Sync instance, many typeclass constraints can be relaxed.

Copy link
Member

@ChristopherDavenport ChristopherDavenport left a comment

This is all in main. Because IO still satisfies all contracts so there are no changes in tests?

Copy link
Member

@rossabaker rossabaker left a comment

Needs a scalafmt.

aeons added 2 commits Dec 8, 2017
Now that the different run methods on Streams just require a Sync instance, many typeclass constraints can be relaxed.
@aeons aeons force-pushed the aeons:relax-typeclass-constraints branch to 4df4261 Dec 11, 2017
@aeons aeons merged commit 5a50077 into http4s:master Dec 11, 2017
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@aeons aeons deleted the aeons:relax-typeclass-constraints branch Dec 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.