Skip to content

Commit 009ffdb

Browse files
committed
lib: remove deprecated 'sys'
this appears to have been deprecated since node 0.7. perhaps it’s time to remove it?
1 parent 1e8cb08 commit 009ffdb

File tree

6 files changed

+2
-61
lines changed

6 files changed

+2
-61
lines changed

lib/sys.js

Lines changed: 0 additions & 29 deletions
This file was deleted.

node.gyp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,6 @@
8787
'lib/_stream_passthrough.js',
8888
'lib/_stream_wrap.js',
8989
'lib/string_decoder.js',
90-
'lib/sys.js',
9190
'lib/timers/promises.js',
9291
'lib/timers.js',
9392
'lib/tls.js',

test/fixtures/GH-7131/a.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
require('sys'); // Builtin should not show up in module.children array.
2+
require('util'); // Builtin should not show up in module.children array.
33
require('./b'); // This should.
44
require('./b'); // This should not.
55
module.exports = module.children.slice();

test/parallel/test-internal-module-require.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,6 @@ const expectedPublicModules = new Set([
6060
'repl',
6161
'stream',
6262
'string_decoder',
63-
'sys',
6463
'timers',
6564
'tls',
6665
'trace_events',

test/parallel/test-module-builtin.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ const { builtinModules } = require('module');
55

66
// Includes modules in lib/ (even deprecated ones)
77
assert(builtinModules.includes('http'));
8-
assert(builtinModules.includes('sys'));
8+
assert(builtinModules.includes('util'));
99

1010
// Does not include internal modules
1111
assert.deepStrictEqual(

test/parallel/test-sys.js

Lines changed: 0 additions & 28 deletions
This file was deleted.

0 commit comments

Comments
 (0)