From ba45a7961c83342857d3add3e413abe88b796b85 Mon Sep 17 00:00:00 2001 From: Andrew Branch Date: Wed, 9 Nov 2022 15:22:26 -0800 Subject: [PATCH] Clean up --- src/compiler/moduleNameResolver.ts | 2 -- src/compiler/utilities.ts | 2 -- src/testRunner/unittests/config/configurationExtension.ts | 1 - 3 files changed, 5 deletions(-) diff --git a/src/compiler/moduleNameResolver.ts b/src/compiler/moduleNameResolver.ts index b24ba667a1f..6200d3cf73e 100644 --- a/src/compiler/moduleNameResolver.ts +++ b/src/compiler/moduleNameResolver.ts @@ -1748,7 +1748,6 @@ export function getEntrypointsFromPackageJsonInfo( } let entrypoints: string[] | undefined; - // TODO: + Json? const extensions = Extensions.TypeScript | Extensions.Declaration | (resolveJs ? Extensions.JavaScript : 0); const features = getDefaultNodeResolutionFeatures(options); const requireState = getTemporaryModuleResolutionState(cache?.getPackageJsonInfoCache(), host, options); @@ -1960,7 +1959,6 @@ function loadNodeModuleFromDirectoryWorker(extensions: Extensions, candidate: st } // Even if extensions is DtsOnly, we can still look up a .ts file as a result of package.json "types" - // TODO: what? Why even bother with DtsOnly? const expandedExtensions = extensions === Extensions.Declaration ? Extensions.TypeScript | Extensions.Declaration : extensions; // Don't do package.json lookup recursively, because Node.js' package lookup doesn't. diff --git a/src/compiler/utilities.ts b/src/compiler/utilities.ts index 2447c730647..d4bb535ceef 100644 --- a/src/compiler/utilities.ts +++ b/src/compiler/utilities.ts @@ -7778,8 +7778,6 @@ const allSupportedExtensionsWithJson: readonly Extension[][] = [...allSupportedE export const supportedDeclarationExtensions: readonly Extension[] = [Extension.Dts, Extension.Dcts, Extension.Dmts]; /** @internal */ export const supportedTSImplementationExtensions: readonly Extension[] = [Extension.Ts, Extension.Cts, Extension.Mts, Extension.Tsx]; -/** @internal */ -export const supportedImplementationExtensions: readonly Extension[] = [...supportedTSImplementationExtensions, ...supportedJSExtensionsFlat]; /** @internal */ export function getSupportedExtensions(options?: CompilerOptions): readonly Extension[][]; diff --git a/src/testRunner/unittests/config/configurationExtension.ts b/src/testRunner/unittests/config/configurationExtension.ts index bd36b2a155d..8434eb350cf 100644 --- a/src/testRunner/unittests/config/configurationExtension.ts +++ b/src/testRunner/unittests/config/configurationExtension.ts @@ -232,7 +232,6 @@ describe("unittests:: config:: configurationExtension", () => { function testSuccess(name: string, entry: string, expected: ts.CompilerOptions, expectedFiles: string[]) { expected.configFilePath = entry; it(name, () => { - console.log(name); const parsed = getParseCommandLine(entry); assert(!parsed.errors.length, ts.flattenDiagnosticMessageText(parsed.errors[0] && parsed.errors[0].messageText, "\n")); assert.deepEqual(parsed.options, expected);