Skip to content

test: increase read file abort coverage - #36716

Closed
shootermv wants to merge 1 commit into
nodejs:masterfrom
shootermv:increase-abort-readfile-test-coverage
Closed

test: increase read file abort coverage#36716
shootermv wants to merge 1 commit into
nodejs:masterfrom
shootermv:increase-abort-readfile-test-coverage

Conversation

@shootermv

Copy link
Copy Markdown
Contributor

No description provided.

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Jan 1, 2021
@shootermv
shootermvforce-pushed the increase-abort-readfile-test-coverage branch from d347db7 to 5a80156CompareJanuary 1, 2021 06:43

@benjamingrbenjamingr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, I would make the comment shorter, thanks for this :)

@benjamingrbenjamingr added the request-ci Add this label to start a Jenkins CI on a PR. label Jan 1, 2021
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 1, 2021

@LxxyxLxxyx left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM except one nit, you can include a link in the commit message, to let us know which part of the code test coverage has been improved, like this pull request: #36628

@benjamingrbenjamingr added the request-ci Add this label to start a Jenkins CI on a PR. label Jan 2, 2021
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 2, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@benjamingr

Copy link
Copy Markdown
Member

Thanks! 🙏

Landed in 8c3cdce 🚀

benjamingr pushed a commit that referenced this pull request Jan 27, 2021
PR-URL: #36716
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
targos pushed a commit that referenced this pull request Feb 2, 2021
PR-URL: #36716
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@targostargos mentioned this pull request Feb 2, 2021
targos pushed a commit that referenced this pull request May 1, 2021
PR-URL: #36716
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@danielleadamsdanielleadams mentioned this pull request May 3, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@shootermv@nodejs-github-bot@benjamingr@jasnell@Trott@Lxxyx