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

Jump to bottom

v2 - re-add NewProxyReader? #114

Closed
zachgersh opened this issue Oct 4, 2017 · 2 comments
Closed

v2 - re-add NewProxyReader? #114

zachgersh opened this issue Oct 4, 2017 · 2 comments

Comments

@zachgersh
Copy link

zachgersh commented Oct 4, 2017 •
edited

Really appreciate the amount of work that has gone into pb! Wanted to ask, were there plans to put NewProxyReader back into v2? Now that v2 doesn't conform to io.Writer we are pretty much stuck (since you can't wire up your own io.TeeReader
@zachgersh zachgersh changed the title v2 - re-add proxy reader? v2 - re-add ProxyReader? Oct 4, 2017
@zachgersh zachgersh changed the title v2 - re-add ProxyReader? v2 - re-add NewProxyReader? Oct 4, 2017
@cheggaaa
Copy link
Owner

Done :)
(50a44ed)
v2.0.5 published

cheggaaa added a commit that referenced this issue Oct 17, 2017
@cheggaaa
#114 add proxy reader
50a44ed
@zachgersh
Copy link
Author

@cheggaaa 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cheggaaa @zachgersh