-
-
Notifications
You must be signed in to change notification settings - Fork 735
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
Fix #992 Bind the socket event listener to the request object #993
Conversation
Oh it appears its just node.js 0.10 that is failing, does this need to be supported? Also if these changes failed 8.2.1 probably failed as well. |
Using nvm to switch over to node 0.10 and then running: I've ascertained that the same test has failed on node 0.10 since v8.1.0 and v8.0.0 won't run entirely due to a forEach issue. The test that's failing is |
); | ||
r.end(); | ||
} | ||
if (process.versions.node >= '0.11' ) { |
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 was the only line added
No they don’t, see the supported versions at https://github.com/node-nock/nock#node-version-support Your PR is great as is though, I’ll make a follow up PR which removes testing the old Node versions and any references to them in the code. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue and add a reference to this one if it’s related. Thank you! |
Backport of 50859e5 from 9.0.x to 8.2.x. Fixes compatibility with Axios. Fixes #992.