feat: implement createReadStream and createWriteStream on FileHandle
#1076
Conversation
|
Well, that's embarrassing: the first version of this that I submitted didn't work, due to another issue: the In my second commit, I've aligned that behavior with the Node API (which makes my first commit work), and in my third commit, I've added a test for this. Let me know if you want me to rebase, or handle any of this differently. |
G-Rath
changed the title
createReadStream and createWriteStream on FileHandle
would you be ok doing a dedicated PR for that? it sounds like it's a clean bugfix that can get its own version and changelog entry |
|
Done: #1077 I'll wait for that to be merged then rebase this. |
graue
force-pushed
the
fh-createwritestream
branch
from
21f729d
to
c550f22
Compare
|
Rebased! |
| @@ -33,6 +33,14 @@ export class FileHandle implements IFileHandle { | |||
| return promisify(this.fs, 'fdatasync')(this.fd); | |||
| } | |||
|
|
|||
| createReadStream(options: opts.IFileHandleReadStreamOptions): IReadStream { | |||
There was a problem hiding this comment.
would it be possible to get some basic tests for these?
Closes #1063
Thanks for making this library! My test needed these methods to exist so I took a stab at adding support.