Skip to content

Configure artifact retention: 14 days for PRs, 90 days for push#522

Merged
ikkentim merged 2 commits into
mainfrom
copilot/configure-artifact-retention
May 4, 2026
Merged

Configure artifact retention: 14 days for PRs, 90 days for push#522
ikkentim merged 2 commits into
mainfrom
copilot/configure-artifact-retention

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 4, 2026

Artifacts from PR workflows were retained indefinitely at GitHub's default (90 days), same as push-triggered builds, despite PRs needing a much shorter window.

Changes

  • Added retention-days to every actions/upload-artifact step across all 6 CI workflow files
  • Uses a conditional expression to apply different retention per trigger event:
retention-days: ${{ github.event_name == 'pull_request' && 14 || 90 }}

Affected workflows: dotnet-component.yml, dotnet-legacy.yml, component-linux.yml, component-win64.yml, plugin-legacy-linux.yml, plugin-legacy-win32.yml

Copilot AI linked an issue May 4, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Configure artifact retention for workflows Configure artifact retention: 14 days for PRs, 90 days for push May 4, 2026
Copilot AI requested a review from ikkentim May 4, 2026 21:45
@ikkentim ikkentim marked this pull request as ready for review May 4, 2026 21:47
@ikkentim ikkentim merged commit 345fea1 into main May 4, 2026
4 checks passed
@ikkentim ikkentim deleted the copilot/configure-artifact-retention branch May 4, 2026 21:51
ikkentim added a commit that referenced this pull request May 10, 2026
…nt-npcs

* origin/main:
  Fix unmanaged→managed event marshalling order and return conversion in source generator (#525)
  Refactor: Convert player classes to ECS components (#513)
  Implement rich command processor (#523)
  Configure artifact retention: 14 days for PRs, 90 days for push (#522)
  docs: Remove 'open.mp only' restriction from IsGhostModeEnabled (#520)
  fix: Windows build scripts fail correctly on build errors (#518)
  Fix 'cdecl' attribute ignored warnings in Linux x86_64 builds (#516)
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.

Configure artifact retention for workflows

2 participants