FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

test: improve coverage of lib/events.js by pd4d10 · Pull Request #38582 · nodejs/node · GitHub

/ node Public

test: improve coverage of lib/events.js - #38582

Closed
pd4d10 wants to merge 1 commit into
nodejs:masterfrom
pd4d10:patch-12
Closed

test: improve coverage of lib/events.js#38582
pd4d10 wants to merge 1 commit into
nodejs:masterfrom
pd4d10:patch-12

Conversation

pd4d10 commented May 7, 2021
edited
Loading

Copy link
Copy Markdown
Contributor

Also rename the file (event to events) to keep consistant with other test files.

github-actions Bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels May 7, 2021

Copy link
Copy Markdown
Collaborator

Lxxyx added request-ci Add this label to start a Jenkins CI on a PR. and removed needs-ci PRs that need a full CI run. labels Jun 10, 2021
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 10, 2021

Copy link
Copy Markdown
Collaborator

Lxxyx added author ready PRs that have at least one approval, no outstanding review comments, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Jun 11, 2021
github-actions Bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 11, 2021

Copy link
Copy Markdown
Contributor

Landed in bf2473d...a596fdf

github-actions Bot closed this Jun 11, 2021
nodejs-github-bot pushed a commit that referenced this pull request Jun 11, 2021
PR-URL: #38582
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
targos pushed a commit that referenced this pull request Jun 11, 2021
PR-URL: #38582
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
pd4d10 deleted the patch-12 branch June 11, 2021 08:46
danielleadams mentioned this pull request Jun 14, 2021
danielleadams pushed a commit that referenced this pull request Jun 17, 2021
PR-URL: #38582
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
richardlau pushed a commit that referenced this pull request Jul 19, 2021
PR-URL: #38582
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
richardlau pushed a commit that referenced this pull request Jul 20, 2021
PR-URL: #38582
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
richardlau mentioned this pull request Jul 20, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL