From 84e80cea7e53b85af543663db34bc71d690c1af0 Mon Sep 17 00:00:00 2001 From: "Michael B. Gale" Date: Wed, 28 Oct 2026 12:35:56 +0000 Subject: [PATCH] Avoid producing the same-ish message twice --- lib/entry-points.js | 1 + src/languages/traced.ts | 4 ++++ 2 files changed, 5 insertions(+) diff --git a/lib/entry-points.js b/lib/entry-points.js index e025efbdd..50b1cb5f9 100644 --- a/lib/entry-points.js +++ b/lib/entry-points.js @@ -194001,6 +194001,7 @@ async function checkRunnerSupportsTracedAnalysis(action, config, codeql) { "macOS", version ); + return; } const xcodeVer = xcodeVersion(action.logger); if (xcodeVer !== void 0 && xcodeVer.major >= UNSUPPORTED_XCODE_FOR_TRACED) { diff --git a/src/languages/traced.ts b/src/languages/traced.ts index 85f9ddc1b..35347a0c5 100644 --- a/src/languages/traced.ts +++ b/src/languages/traced.ts @@ -156,6 +156,10 @@ export async function checkRunnerSupportsTracedAnalysis( "macOS", version, ); + + // Return early so that we don't end up producing more-or-less the same log message/diagnostic + // twice. The ones for macOS already comment on the Xcode requirement. + return; } // Determining whether the Xcode version is supported only makes sense on macOS, so we only do it