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

Remove Streaming and StreamingT #910

Merged
merged 2 commits into from
Mar 3, 2016
Merged

Conversation

ceedubs
Copy link
Contributor

@ceedubs ceedubs commented Mar 3, 2016

This resolves #858 and is based on a discussion at dinner after the first day of the typelevel summit including @travisbrown, @non, @stew, and several others.

Per the discussion, Streaming will be moved to the dogs project.
StreamingT will be removed for now, and we can consider adding it to
dogs in the future if people find themselves wanting it.

This is based on typelevel#858 and a discussion at dinner after the first day of
the typelevel summit.
This is based on typelevel#858 and a discussion at dinner after the first day of the typelevel summit.

Per the discussion, Streaming will be moved to the dogs project.
StreamingT will be removed for now, and we can consider adding it to
dogs in the future if people find themselves wanting it.
@codecov-io
Copy link

Current coverage is 88.77%

Merging #910 into master will decrease coverage by -0.82% as of d5fde3f

@@            master    #910   diff @@
======================================
  Files          181     178     -3
  Stmts         2423    2121   -302
  Branches        76      42    -34
  Methods          0       0       
======================================
- Hit           2171    1883   -288
  Partial          0       0       
+ Missed         252     238    -14

Review entire Coverage Diff as of d5fde3f

Powered by Codecov. Updated on successful CI builds.

@stew
Copy link
Contributor

stew commented Mar 3, 2016

👍

2 similar comments
@travisbrown
Copy link
Contributor

👍

@adelbertc
Copy link
Contributor

👍

adelbertc added a commit that referenced this pull request Mar 3, 2016
Remove Streaming and StreamingT
@adelbertc adelbertc merged commit 253dc0d into typelevel:master Mar 3, 2016
@ceedubs ceedubs deleted the rm-streaming branch March 3, 2016 19:00
This was referenced Apr 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move Streaming(T) to a separate module
5 participants