felixge / felixge/node-combined-stream

Suggestion to accept list of streams as an argument

Open
#50 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
145
Forks
35
PR merge metrics
No merged PRs in 30d

Description

Hello. I am wondering - why isn't there an option to directly reference streams array in the constructor?

So intead of
```
const combinedStream = new CombinedStream();

readStreams.forEach((readStream) => {
combinedStream.append(readStream);
});
```

I could use
```
const combinedStream = new CombinedStream({ streams: readStreams });
```

Seems like a nice feature to have

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.