Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

[pr] Upgrade to Node v10 #112

Merged
merged 2 commits into from Feb 26, 2019
Merged

[pr] Upgrade to Node v10 #112

merged 2 commits into from Feb 26, 2019

Conversation

stephen-palmer
Copy link
Contributor

No description provided.

… listener (or pipe) on client socket connections to drain the incoming byte buffer, the 'close' event will never be triggered after calling socket.end();

- Updated mocha version
Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ A review job has been created and sent to the PullRequest network.


Check the status or cancel PullRequest code review here - or - cancel by adding [!pr] to the title of the pull request.

@coveralls
Copy link

coveralls commented Feb 26, 2019

Coverage Status

Coverage increased (+0.7%) to 95.0% when pulling 17c76de on dev/node-v10-upgrade into 970e1f1 on master.

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me.


Was this helpful? Yes | No

@stephen-palmer stephen-palmer merged commit d6d9fb6 into master Feb 26, 2019
Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 Message
📚 It looks like the description for this pull request is either blank or very short. Adding a high-level summary will help our reviewers provide better feedback. Feel free to include questions for PullRequest reviewers and make specific feedback requests.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants