-
Notifications
You must be signed in to change notification settings - Fork 30.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
doc: update links and names for DevTools Protocol #20111
Conversation
Please, add 👍 here to approve fast-tracking. |
@@ -128,7 +128,7 @@ Activate inspector on host:port. Default is 127.0.0.1:9229. | |||
|
|||
V8 inspector integration allows tools such as Chrome DevTools and IDEs to debug | |||
and profile Node.js instances. The tools attach to Node.js instances via a | |||
tcp port and communicate using the [Chrome Debugging Protocol][]. | |||
tcp port and communicate using the [Chrome DevTools Protocol][]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small nit: for consistency, also update
Lines 110 to 117 in a43e889
.It Fl -inspect Ns = Ns Ar [host:]port | |
Activate inspector on | |
.Ar host:port . | |
Default is | |
.Sy 127.0.0.1:9229 . | |
.Pp | |
V8 Inspector integration allows attaching Chrome DevTools and IDEs to Node.js instances for debugging and profiling. | |
It uses the Chrome Debugging Protocol. |
CI-lite: https://ci.nodejs.org/job/node-test-pull-request-lite/541/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 to fast-tracking
PR-URL: #20111 Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Eugene Ostroukhov <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Richard Lau <[email protected]>
Landed in c98eeb6 |
PR-URL: #20111 Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Eugene Ostroukhov <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Richard Lau <[email protected]>
Checklist
make -j4 test
(UNIX), orvcbuild test
(Windows) passesThe old URL is now redirected to this URL.