Skip to content

Commit

Permalink
debugger: rename internal library for clarity
Browse files Browse the repository at this point in the history
When I moved these files from node-inspect to Node.js core, I put them
in lib/internal/inspector. That was a mistake. They should be in
lib/internal/debugger.

PR-URL: #39080
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
  • Loading branch information
Trott authored and danielleadams committed Jun 21, 2021
1 parent 7b026d8 commit e4f802d
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 6 deletions.
Expand Up @@ -39,8 +39,8 @@ const console = require('internal/console/global');

const { 0: InspectClient, 1: createRepl } =
[
require('internal/inspector/inspect_client'),
require('internal/inspector/inspect_repl'),
require('internal/debugger/inspect_client'),
require('internal/debugger/inspect_repl'),
];

const debuglog = util.debuglog('inspect');
Expand Down
File renamed without changes.
File renamed without changes.
2 changes: 1 addition & 1 deletion lib/internal/main/inspect.js
Expand Up @@ -13,5 +13,5 @@ markBootstrapComplete();

// Start the debugger agent.
process.nextTick(() => {
require('internal/inspector/_inspect').start();
require('internal/debugger/_inspect').start();
});
6 changes: 3 additions & 3 deletions node.gyp
Expand Up @@ -143,6 +143,9 @@
'lib/internal/crypto/webcrypto.js',
'lib/internal/crypto/x509.js',
'lib/internal/constants.js',
'lib/internal/debugger/_inspect.js',
'lib/internal/debugger/inspect_client.js',
'lib/internal/debugger/inspect_repl.js',
'lib/internal/dgram.js',
'lib/internal/dns/promises.js',
'lib/internal/dns/utils.js',
Expand All @@ -166,9 +169,6 @@
'lib/internal/heap_utils.js',
'lib/internal/histogram.js',
'lib/internal/idna.js',
'lib/internal/inspector/_inspect.js',
'lib/internal/inspector/inspect_client.js',
'lib/internal/inspector/inspect_repl.js',
'lib/internal/inspector_async_hook.js',
'lib/internal/js_stream_socket.js',
'lib/internal/legacy/processbinding.js',
Expand Down

0 comments on commit e4f802d

Please sign in to comment.