Skip to content

Commit d97bb67

Browse files
TypeScript Botjakebailey
TypeScript Bot
andauthored
🤖 Pick PR #53174 (Remove old references to Node v4, s...) into release-5.0 (#53189)
Co-authored-by: Jake Bailey <[email protected]>
1 parent 6e59874 commit d97bb67

File tree

5 files changed

+3
-26
lines changed

5 files changed

+3
-26
lines changed

Diff for: ‎src/compiler/sys.ts

+1-18
Original file line numberDiff line numberDiff line change
@@ -1449,21 +1449,6 @@ declare const global: any;
14491449
declare const __filename: string;
14501450
declare const __dirname: string;
14511451

1452-
export function getNodeMajorVersion(): number | undefined {
1453-
if (typeof process === "undefined") {
1454-
return undefined;
1455-
}
1456-
const version: string = process.version;
1457-
if (!version) {
1458-
return undefined;
1459-
}
1460-
const dot = version.indexOf(".");
1461-
if (dot === -1) {
1462-
return undefined;
1463-
}
1464-
return parseInt(version.substring(1, dot));
1465-
}
1466-
14671452
// TODO: GH#18217 this is used as if it's certainly defined in many places.
14681453
// eslint-disable-next-line prefer-const
14691454
export let sys: System = (() => {
@@ -1493,8 +1478,6 @@ export let sys: System = (() => {
14931478
from?(input: string, encoding?: string): any;
14941479
} = require("buffer").Buffer;
14951480

1496-
const nodeVersion = getNodeMajorVersion();
1497-
const isNode4OrLater = nodeVersion! >= 4;
14981481
const isLinuxOrMacOs = process.platform === "linux" || process.platform === "darwin";
14991482

15001483
const platform: string = _os.platform();
@@ -1508,7 +1491,7 @@ export let sys: System = (() => {
15081491
// Note that if we ever emit as files like cjs/mjs, this check will be wrong.
15091492
const executingFilePath = __filename.endsWith("sys.js") ? _path.join(_path.dirname(__dirname), "__fake__.js") : __filename;
15101493

1511-
const fsSupportsRecursiveFsWatch = isNode4OrLater && (process.platform === "win32" || process.platform === "darwin");
1494+
const fsSupportsRecursiveFsWatch = process.platform === "win32" || process.platform === "darwin";
15121495
const getCurrentDirectory = memoize(() => process.cwd());
15131496
const { watchFile, watchDirectory } = createSystemWatchFunctions({
15141497
pollingWatchFileWorker: fsWatchFileWorker,

Diff for: ‎src/tsserver/nodeServer.ts

+1-4
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ import {
1111
DirectoryWatcherCallback,
1212
FileWatcher,
1313
getDirectoryPath,
14-
getNodeMajorVersion,
1514
getRootLength,
1615
JsTyping,
1716
LanguageServiceMode,
@@ -299,9 +298,7 @@ export function initializeNodeSystem(): StartInput {
299298

300299
const libDirectory = getDirectoryPath(normalizePath(sys.getExecutingFilePath()));
301300

302-
const nodeVersion = getNodeMajorVersion();
303-
// use watchGuard process on Windows when node version is 4 or later
304-
const useWatchGuard = process.platform === "win32" && nodeVersion! >= 4;
301+
const useWatchGuard = process.platform === "win32";
305302
const originalWatchDirectory: ServerHost["watchDirectory"] = sys.watchDirectory.bind(sys);
306303
const logger = createLogger();
307304

Diff for: ‎src/tsserver/server.ts

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import {
22
Debug,
3-
getNodeMajorVersion,
43
setStackTraceLimit,
54
sys,
65
version,
@@ -30,7 +29,7 @@ function start({ args, logger, cancellationToken, serverMode, unknownServerMode,
3029
logger.info(`Starting TS Server`);
3130
logger.info(`Version: ${version}`);
3231
logger.info(`Arguments: ${args.join(" ")}`);
33-
logger.info(`Platform: ${platform} NodeVersion: ${getNodeMajorVersion()} CaseSensitive: ${sys.useCaseSensitiveFileNames}`);
32+
logger.info(`Platform: ${platform} NodeVersion: ${process.version} CaseSensitive: ${sys.useCaseSensitiveFileNames}`);
3433
logger.info(`ServerMode: ${serverMode} hasUnknownServerMode: ${unknownServerMode}`);
3534

3635
setStackTraceLimit();

Diff for: ‎tests/baselines/reference/api/tsserverlibrary.d.ts

-1
Original file line numberDiff line numberDiff line change
@@ -8286,7 +8286,6 @@ declare namespace ts {
82868286
negative: boolean;
82878287
base10Value: string;
82888288
}
8289-
function getNodeMajorVersion(): number | undefined;
82908289
enum FileWatcherEventKind {
82918290
Created = 0,
82928291
Changed = 1,

Diff for: ‎tests/baselines/reference/api/typescript.d.ts

-1
Original file line numberDiff line numberDiff line change
@@ -4311,7 +4311,6 @@ declare namespace ts {
43114311
negative: boolean;
43124312
base10Value: string;
43134313
}
4314-
function getNodeMajorVersion(): number | undefined;
43154314
enum FileWatcherEventKind {
43164315
Created = 0,
43174316
Changed = 1,

0 commit comments

Comments
 (0)