Skip to content

fix(ci): upload example assets only for releases - #1608

Merged
mrousavy merged 1 commit into
perf/cleanup-cifrom
perf/cleanup-release
Sep 6, 2026
Merged

fix(ci): upload example assets only for releases#1608
mrousavy merged 1 commit into
perf/cleanup-cifrom
perf/cleanup-release

Conversation

@mrousavy

@mrousavy mrousavy commented Sep 5, 2026

Copy link
Copy Markdown
Member

The example Release workflows currently build successfully on pull requests, then try to upload a release asset without a release upload URL. Restrict asset uploads to published release events and use the GitHub CLI in place of the unmaintained upload action. Published releases keep the same asset names; pull requests still exercise the builds and packaging.

Validated both workflows with actionlint and executed the exact upload scripts against temporary APK/ZIP fixtures with a stubbed GitHub CLI, checking renamed assets and arguments. No release was published. CI runs for this draft are intentionally cancelled to conserve Actions minutes.

@vercel

vercel Bot commented Sep 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
nitro-docs Ready Ready Preview Sep 5, 2026 4:05pm UTC

Request Review

@mrousavy
mrousavy marked this pull request as ready for review September 6, 2026 18:59
@mrousavy
mrousavy merged commit 2530e1e into perf/cleanup-ci Sep 6, 2026
10 of 12 checks passed
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