From e8478dd155416bcaf917d70c7f27238d2c3fe1d7 Mon Sep 17 00:00:00 2001 From: Angela P Wen Date: Tue, 25 Oct 2022 14:37:39 -0700 Subject: [PATCH] Remove `CODEQL_EXTRACTOR_GO_BUILD_TRACING` logic --- src/languages.ts | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/src/languages.ts b/src/languages.ts index 372d7d815..fcc1e07c4 100644 --- a/src/languages.ts +++ b/src/languages.ts @@ -45,22 +45,15 @@ export function isTracedLanguage( isGoExtractionReconciliationEnabled: boolean, logger: Logger ): boolean { - if (process.env["CODEQL_EXTRACTOR_GO_BUILD_TRACING"] === "true") { + if ("CODEQL_EXTRACTOR_GO_BUILD_TRACING" in process.env) { logger.warning( - "The CODEQL_EXTRACTOR_GO_BUILD_TRACING environment variable was set to 'true', but it must " + - "be 'on' to enable Go build tracing. Setting it to 'on'." + "The CODEQL_EXTRACTOR_GO_BUILD_TRACING environment variable is deprecated and its behavior is now the default behavior." ); - process.env["CODEQL_EXTRACTOR_GO_BUILD_TRACING"] = "on"; - core.exportVariable("CODEQL_EXTRACTOR_GO_BUILD_TRACING", "on"); } - const shouldTraceGo = - process.env["CODEQL_EXTRACTOR_GO_BUILD_TRACING"] === "on" || - isGoExtractionReconciliationEnabled; - return ( ["cpp", "java", "csharp", "swift"].includes(language) || - (shouldTraceGo && language === Language.go) + (isGoExtractionReconciliationEnabled && language === Language.go) ); }