From b96e3390db93dc46c2c3753a94b87c0d734b0b61 Mon Sep 17 00:00:00 2001 From: "Michael B. Gale" Date: Tue, 20 Oct 2026 18:07:32 +0100 Subject: [PATCH] Change comment for `getConfig` call in `upload-sarif` --- src/upload-sarif-action.ts | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/upload-sarif-action.ts b/src/upload-sarif-action.ts index 52dd12ec0..447314fb1 100644 --- a/src/upload-sarif-action.ts +++ b/src/upload-sarif-action.ts @@ -93,10 +93,8 @@ async function run({ startedAt, logger }: ActionState<["Base", "Logger"]>) { const checkoutPath = actionsUtil.getRequiredInput("checkout_path"); const category = actionsUtil.getOptionalInput("category"); - // Although we wouldn't typically expect there to be a configuration since we don't expect another - // CodeQL Action step to have come before `upload-sarif`, this has previously been the behaviour - // when `combineSarifFilesUsingCLI` called `getConfig` directly. Since the only effect of this would - // be that an existing CLI that was installed gets re-used, we maintain this behaviour. + // Load the configuration from a previous step if there is one. This allows us to reuse the CLI that + // may have been installed by a previous step. const maybeConfig = await getConfig(tempDir, logger); const uploadResults = await postProcessAndUploadSarif( logger,