Skip to content

Commit

Permalink
Remove redundant log messages
Browse files Browse the repository at this point in the history
  • Loading branch information
Henry Mercer committed Dec 22, 2022
1 parent 3224214 commit 59ebabd
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 13 deletions.
5 changes: 1 addition & 4 deletions lib/init-action-post-helper.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/init-action-post-helper.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 1 addition & 8 deletions src/init-action-post-helper.ts
Original file line number Diff line number Diff line change
Expand Up @@ -47,9 +47,6 @@ async function maybeUploadFailedSarif(
logger: Logger
): Promise<UploadFailedSarifResult> {
if (!config.codeQLCmd) {
logger.warning(
"CodeQL command not found. Unable to upload failed SARIF file."
);
return { upload_failed_run_skipped_because: "CodeQL command not found" };
}
const codeql = await getCodeQL(config.codeQLCmd);
Expand All @@ -59,7 +56,6 @@ async function maybeUploadFailedSarif(
codeql
))
) {
logger.debug("Uploading failed SARIF is disabled.");
return { upload_failed_run_skipped_because: "Feature disabled" };
}
const workflow = await getWorkflow();
Expand All @@ -69,9 +65,6 @@ async function maybeUploadFailedSarif(
getUploadInputOrThrow(workflow, jobName, matrix) !== "true" ||
isInTestMode()
) {
logger.debug(
"Won't upload a failed SARIF file since SARIF upload is disabled."
);
return { upload_failed_run_skipped_because: "SARIF upload is disabled" };
}
const category = getCategoryInputOrThrow(workflow, jobName, matrix);
Expand Down Expand Up @@ -148,7 +141,7 @@ export async function run(
);
if (uploadFailedSarifResult.upload_failed_run_skipped_because) {
logger.debug(
"Skipped uploading a failed SARIF file for this CodeQL code scanning run because: " +
"Won't upload a failed SARIF file for this CodeQL code scanning run because: " +
`${uploadFailedSarifResult.upload_failed_run_skipped_because}.`
);
}
Expand Down

0 comments on commit 59ebabd

Please sign in to comment.