Robert Nagy

Results 379 comments of Robert Nagy

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...