Skip to content

ci: draft-then-publish releases to close the empty public window#21

Merged
TMHSDigital merged 1 commit into
mainfrom
ci/draft-then-publish
Jun 5, 2026
Merged

ci: draft-then-publish releases to close the empty public window#21
TMHSDigital merged 1 commit into
mainfrom
ci/draft-then-publish

Conversation

@TMHSDigital
Copy link
Copy Markdown
Owner

create-release now makes a DRAFT, upload-assets matrix attaches all 10 assets to the draft (serialized, no race), and a new publish-release job (needs upload-assets) flips draft to false only after all assets are present. Closes the 1-2 min window where the public saw a partial/empty release. upload-assets sets draft: true explicitly so updates do not publish early. Preserves 5 targets, asset names, 10 assets, checksums, GITHUB_TOKEN, contents: write on release jobs only. Validates via v0.6.1-rc4.

Made with Cursor

Previously create-release published an empty release immediately, leaving a
1-2 minute window where the public saw a partial release while the upload
matrix attached assets.

Now the release is created as a draft, the upload matrix attaches all 10 assets
to the draft (still serialized so there is no create race), and a final
publish-release job flips draft to false only after every asset is present. The
release becomes public in one step with the full asset set.

The upload-assets matrix sets draft: true explicitly because the action's draft
input defaults to false and would otherwise publish the draft early on each
update call.

Preserves the 5 platform targets, asset names, 10 total assets, checksums,
GITHUB_TOKEN, and contents: write only on the release-related jobs.

Co-authored-by: Cursor <cursoragent@cursor.com>
@TMHSDigital TMHSDigital merged commit 26c15ec into main Jun 5, 2026
8 checks passed
@TMHSDigital TMHSDigital deleted the ci/draft-then-publish branch June 5, 2026 00:48
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