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

lib: refactor internal debug variable by gdccwxx · Pull Request #40364 · nodejs/node · GitHub

/ node Public

lib: refactor internal debug variable - #40364

Closed
gdccwxx wants to merge 1 commit into
nodejs:masterfrom
gdccwxx:lib/refactor-internal-debug-variable
Closed

lib: refactor internal debug variable#40364
gdccwxx wants to merge 1 commit into
nodejs:masterfrom
gdccwxx:lib/refactor-internal-debug-variable

Conversation

gdccwxx commented Oct 7, 2021

Copy link
Copy Markdown
Contributor
  • refractor internal/debugger/inspect_client.js to use let.
  • move variable into each for loop

nodejs-github-bot added debugger Issues and PRs related to the debugger subsystem. needs-ci PRs that need a full CI run. labels Oct 7, 2021

Lxxyx commented Oct 7, 2021

Copy link
Copy Markdown
Member

nit: The pull request title should start with lib:

Lxxyx added request-ci Add this label to start a Jenkins CI on a PR. and removed request-ci Add this label to start a Jenkins CI on a PR. labels Oct 7, 2021
gdccwxx changed the title Lib/refactor internal debug variable lib/refactor internal debug variable Oct 7, 2021
gdccwxx changed the title lib/refactor internal debug variable lib: refactor internal debug variable Oct 7, 2021
move variable into each for loop
gdccwxx force-pushed the lib/refactor-internal-debug-variable branch from 2f6e1e1 to 604d0d4 Compare October 7, 2021 14:53
Lxxyx added the request-ci Add this label to start a Jenkins CI on a PR. label Oct 7, 2021
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 7, 2021

Copy link
Copy Markdown
Collaborator

Ayase-252 added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Oct 8, 2021
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 eafdeab...ac4f5e2

github-actions Bot closed this Oct 9, 2021
nodejs-github-bot pushed a commit that referenced this pull request Oct 9, 2021
move variable into each for loop

PR-URL: #40364
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
targos pushed a commit that referenced this pull request Oct 13, 2021
move variable into each for loop

PR-URL: #40364
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.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. debugger Issues and PRs related to the debugger subsystem. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL