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

test: fix "test/common/debugger" identify async function by gdccwxx · Pull Request #40348 · nodejs/node · GitHub

/ node Public

test: fix "test/common/debugger" identify async function - #40348

Closed
gdccwxx wants to merge 2 commits into
nodejs:masterfrom
gdccwxx:test/debugger-async-function
Closed

test: fix "test/common/debugger" identify async function#40348
gdccwxx wants to merge 2 commits into
nodejs:masterfrom
gdccwxx:test/debugger-async-function

Conversation

gdccwxx commented Oct 6, 2021

Copy link
Copy Markdown
Contributor

test: use async to identify function

waitForInitialBreak() return a promise function, should use async to identify it

nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Oct 6, 2021
Comment thread test/common/debugger.js Outdated
Ayase-252 added author ready PRs that have at least one approval, no outstanding review comments, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Oct 8, 2021
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 8, 2021

This comment has been minimized.

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 9, 2021
github-actions Bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 9, 2021

github-actions Bot commented Oct 9, 2021

Copy link
Copy Markdown
Contributor

Landed in eaa5957...24f045d

github-actions Bot closed this Oct 9, 2021
nodejs-github-bot pushed a commit that referenced this pull request Oct 9, 2021
PR-URL: #40348
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
nodejs-github-bot pushed a commit that referenced this pull request Oct 9, 2021
PR-URL: #40348
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
targos pushed a commit that referenced this pull request Oct 13, 2021
PR-URL: #40348
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
targos pushed a commit that referenced this pull request Oct 13, 2021
PR-URL: #40348
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
richardlau mentioned this pull request Oct 18, 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. needs-ci PRs that need a full CI run. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL