Robert Nagy
Robert Nagy
Updated test
Fails in node < 0.4. Is that a problem?
Is there anything more I can do here?
The coveralls and node < 4 are a bit out of my expertise.
> you can help me understand the changes in this PR and/or how you've been manually testing the changes, which I can help replicate into automated tests. So the problem...
Another way to phrase it. This PR fixes so that ‘isFinished’ is never true before ‘onFinished’ has been called.
> So I had originally saw your multiple commit iterations, and there is the http/2 issue. Were you testing the changes by hand? If so, can you help me replicate...
This is also relevant https://github.com/nodejs/node/pull/28621
https://github.com/nodejs/node/pull/28681
> We are reserving the nodejs scope for internal /core usage. I'm -1 for using it for external usage. What's the definition of internal vs external? Whether or not it...