Visitar URL original
Use `shouldSkipSarifUpload` in `analyze-action.ts` · github/codeql-action@7fa4518 · GitHub
Skip to content

Commit 7fa4518

Browse files
committed
Use shouldSkipSarifUpload in analyze-action.ts
1 parent a69c7cb commit 7fa4518

2 files changed

Lines changed: 8 additions & 4 deletions

File tree

‎lib/entry-points.js‎

Lines changed: 4 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎src/analyze-action.ts‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -434,8 +434,10 @@ async function run(action: ActionState<["Base", "Logger", "Env", "Actions"]>) {
434434
}
435435

436436
// We don't upload results in test mode, so don't wait for processing
437-
if (util.isInTestMode()) {
438-
logger.debug("In test mode. Waiting for processing is disabled.");
437+
if (util.shouldSkipSarifUpload()) {
438+
logger.debug(
439+
"SARIF upload is disabled and nothing was uploaded, so we do not need to wait for processing to finish.",
440+
);
439441
} else if (
440442
uploadResults?.[analyses.AnalysisKind.CodeScanning] !== undefined &&
441443
actionsUtil.getRequiredInput("wait-for-processing") === "true"

0 commit comments

Comments
 (0)