-
Notifications
You must be signed in to change notification settings - Fork 342
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
feat: missing Read and Write methods #245
Conversation
5a9ba53
to
e681e29
Compare
marking as ready, this can be reviewed and merged, and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is really thorough, and is looking 💯. I've left a few comments about some minor things, but overall this PR is great!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sweet! Thanks so much!
bors r+ |
245: feat: missing Read and Write methods r=yoshuawuyts a=dignifiedquire Ref: #131 - [x] Read::by_ref - [x] Read::bytes - [x] Read::chain - [x] Read::take - [ ] Write::by_ref - [ ] ~~Write::write_fmt~~ postponed until #247 is solved Needs fixing: - [x] `BufRead` for `Take` - [x] `BufRead` for `Chain` - [ ] `by_ref` conflict between `Read` and `Write`, unable to add both, as they conflict, and the current state of things does not allow to differentiate between the two. Co-authored-by: dignifiedquire <dignifiedquire@users.noreply.github.com>
Build succeeded
|
Ref: #131
Write::write_fmtpostponed until [tracking] fmt #247 is solvedNeeds fixing:
BufRead
forTake
BufRead
forChain
by_ref
conflict betweenRead
andWrite
, unable to add both, as they conflict, and the current state of things does not allow to differentiate between the two.