Skip to content

chore(ci): cap Actions artifact retention at 10 days - #1

Merged
sravinet merged 1 commit into
masterfrom
chore/artifact-retention-10d
Aug 29, 2026
Merged

chore(ci): cap Actions artifact retention at 10 days#1
sravinet merged 1 commit into
masterfrom
chore/artifact-retention-10d

Conversation

@sravinet

Copy link
Copy Markdown

What

Sets retention-days: 10 on every actions/upload-artifact step in this repo (1 step(s) across 1 workflow file(s)).

  • check-dist.yml

Why

Part of an org-wide pass. A survey of all 194 CaptainEmpower repos found 4,240 live artifacts holding 1.88 GiB, nearly all of it long past any useful review window. Capping retention at 10 days keeps artifacts around for the period people actually pull them and stops storage growing without bound.

Artifacts older than 10 days were deleted in the same pass.

How it was made

Applied by a transformer run against all 110 upload-artifact workflows in the org and validated there: every upload step parses back to retention-days: 10, no other key changes value, and re-running is a no-op. It handles the three shapes — an existing retention-days (value rewritten), a with: block without one (key added), and a step with no with: block (both added) — and stays clear of path: | block scalars.

Only the retention value changes; no workflow logic is touched.

Sets retention-days: 10 on 1 upload-artifact step(s) across 1 workflow file(s). Part of an org-wide pass; artifacts older than 10 days were deleted in the same pass.
@sravinet
sravinet marked this pull request as ready for review August 29, 2026 19:33
@sravinet
sravinet merged commit ec20aaa into master Aug 29, 2026
30 checks passed
@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Cap check-dist artifact retention at 10 days

⚙️ Configuration changes 🕐 Less than 5 minutes

Grey Divider

AI Description

• Limits failed distribution-build artifacts to a 10-day retention window.
• Reduces long-term GitHub Actions storage without changing workflow execution logic.
High-Level Assessment

The explicit per-step retention setting is the appropriate GitHub Actions mechanism and directly supports the organization-wide policy. Repository-level retention defaults were considered less suitable because this change intentionally standardizes each upload-artifact step and makes the policy visible alongside the artifact configuration.

Files changed (1) +1 / -0

Other (1) +1 / -0
check-dist.ymlLimit dist artifact retention to 10 days +1/-0

Limit dist artifact retention to 10 days

• Adds 'retention-days: 10' to the failed distribution-check artifact upload. Artifact creation conditions, contents, and other workflow behavior remain unchanged.

.github/workflows/check-dist.yml

@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can group findings by type and pick your Finding display, from Minimal to Full

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant