Skip to content

Commit cd94990

Browse files
authored
Merge pull request #2079 from github/aeisenberg/artifact-storage
Add a retention period of 7 days
2 parents 3b54300 + f5a8218 commit cd94990

File tree

4 files changed

+12
-4
lines changed

4 files changed

+12
-4
lines changed

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ Note that the only difference between `v2` and `v3` of the CodeQL Action is the
66

77
## [UNRELEASED]
88

9-
No user facing changes.
9+
- Change the retention period for uploaded debug artifacts to 7 days. Previously, this was whatever the repository default was. [#2079](https://github.com/github/codeql-action/pull/2079)
1010

1111
## 3.23.0 - 08 Jan 2024
1212

lib/debug-artifacts.js

Lines changed: 5 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/debug-artifacts.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/debug-artifacts.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,11 @@ export async function uploadDebugArtifacts(
4949
sanitizeArifactName(`${artifactName}${suffix}`),
5050
toUpload.map((file) => path.normalize(file)),
5151
path.normalize(rootDir),
52-
{ continueOnError: true },
52+
{
53+
continueOnError: true,
54+
// ensure we don't keep the debug artifacts around for too long since they can be large.
55+
retentionDays: 7,
56+
},
5357
);
5458
}
5559

0 commit comments

Comments
 (0)