Skip to content

Commit

Permalink
util: move inspect in separate file
Browse files Browse the repository at this point in the history
The inspect function became very big and it's better to handle this
in a separate file.

Backport-PR-URL: #23226
PR-URL: #22845
Reviewed-By: John-David Dalton <[email protected]>
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: James M Snell <[email protected]>
BridgeAR authored and targos committed Oct 7, 2018
1 parent efdb326 commit 77de1be
Showing 4 changed files with 1,103 additions and 1,074 deletions.
1,100 changes: 1,100 additions & 0 deletions lib/internal/util/inspect.js

Large diffs are not rendered by default.

1,074 changes: 1 addition & 1,073 deletions lib/util.js

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions node.gyp
Original file line number Diff line number Diff line change
@@ -161,6 +161,7 @@
'lib/internal/url.js',
'lib/internal/util.js',
'lib/internal/util/comparisons.js',
'lib/internal/util/inspect.js',
'lib/internal/util/inspector.js',
'lib/internal/util/types.js',
'lib/internal/http2/core.js',
2 changes: 1 addition & 1 deletion test/parallel/test-bootstrap-modules.js
Original file line number Diff line number Diff line change
@@ -11,4 +11,4 @@ const list = process.moduleLoadList.slice();

const assert = require('assert');

assert(list.length <= 75, list);
assert(list.length <= 76, list);

0 comments on commit 77de1be

Please sign in to comment.