Skip to content

release.yml: make macOS artifact retention explicit#9

Merged
simonpainter merged 5 commits into
mainfrom
copilot/set-explicit-artifact-retention-again
May 9, 2026
Merged

release.yml: make macOS artifact retention explicit#9
simonpainter merged 5 commits into
mainfrom
copilot/set-explicit-artifact-retention-again

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 9, 2026

The release workflow uploaded the macOS build artifact without an explicit retention policy, leaving lifecycle/cost behavior implicit. This updates the workflow to set a deliberate short retention window.

  • Workflow change

    • Added retention-days: 1 to the macOS actions/upload-artifact@v4 step in .github/workflows/release.yml.
    • Scope is intentionally limited to the uping-macos artifact upload path.
  • Resulting behavior

    • macOS release artifacts now expire on a defined schedule instead of relying on repository/org defaults.
- name: Upload artifact
  uses: actions/upload-artifact@v4
  with:
    name: uping-macos
    path: uping
    retention-days: 1

Copilot AI and others added 3 commits May 9, 2026 18:52
Copilot AI changed the title [WIP] Update release.yml to set explicit artifact retention-days for macos release.yml: make macOS artifact retention explicit May 9, 2026
Copilot AI requested a review from simonpainter May 9, 2026 18:54
@simonpainter simonpainter requested a review from Copilot May 9, 2026 18:59
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR makes the macOS build artifact retention policy explicit in the release workflow, ensuring the artifact lifecycle is intentional rather than relying on repository/org defaults.

Changes:

  • Set retention-days: 1 for the uping-macos artifact upload in the release workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@simonpainter simonpainter marked this pull request as ready for review May 9, 2026 19:03
@simonpainter simonpainter merged commit 8f6937e into main May 9, 2026
2 checks passed
@simonpainter simonpainter deleted the copilot/set-explicit-artifact-retention-again branch May 10, 2026 09:31
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.

release.yml: set explicit artifact retention-days (macos)

3 participants