Skip to content

Introduce CODEQL_ACTION_SKIP_SARIF_UPLOAD #3180

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
redsun82 merged 12 commits into from
Oct 8, 2025
Merged

Introduce CODEQL_ACTION_SKIP_SARIF_UPLOAD #3180

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
e0b9da7
Introduce `CODEQL_ACTION_SKIP_SARIF_UPLOAD`
redsun82 Oct 6, 2025
882667e
Update src/util.ts
redsun82 Oct 6, 2025
11e4034
Clarify comment about SKIP_SARIF_UPLOAD setting
redsun82 Oct 6, 2025
680b070
Specify reason for skipping SARIF upload in logs
redsun82 Oct 6, 2025
25c8db9
Revert "Specify reason for skipping SARIF upload in logs"
redsun82 Oct 6, 2025
00a6e13
Tweak SARIF skipping logs
redsun82 Oct 6, 2025
5dfb610
Merge branch 'main' into redsun82/skip-sarif-upload
redsun82 Oct 7, 2025
86b2ad6
Remove unneeded comment
redsun82 Oct 7, 2025
1b09eb4
Address review
redsun82 Oct 7, 2025
df65651
Merge branch 'main' into redsun82/skip-sarif-upload
redsun82 Oct 7, 2025
d05f225
Tweak comment
redsun82 Oct 8, 2025
1707898
Merge branch 'main' into redsun82/skip-sarif-upload
redsun82 Oct 8, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove unneeded comment
  • Loading branch information
@redsun82
redsun82 committed Oct 7, 2025
commit 86b2ad66467bec6e0e45071af0ff547052830425
1 change: 0 additions & 1 deletion src/upload-lib.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -356,7 +356,6 @@ async function uploadPayload(
): Promise<string> {
logger.info("Uploading results");

// If in test mode we don't want to upload the results,
if (util.shouldSkipSarifUpload()) {
const payloadSaveFile = path.join(
actionsUtil.getTemporaryDirectory(),
Expand Down
Loading