Skip to content

Allow package consumers to verify draft releases - #14

Merged
antono3 merged 1 commit into
mainfrom
codex/draft-release-consumer-access
Oct 11, 2026
Merged

antono3 merged 1 commit into
mainfrom
codex/draft-release-consumer-access

Conversation

@antono3

@antono3 antono3 commented Oct 11, 2026 •

Copy link
Copy Markdown
Collaborator

Draft consumers failed with release not found because the workflow's contents: read token cannot see draft releases. Give the consumer workflow contents: write, which GitHub requires for draft visibility, and document the requirement. The jobs only download assets and upload test logs.

Validation: actionlint and git diff --check passed. Linux and Windows draft consumers passed in run38130244355; public downloads passed in run38130369426 after publication. Release binaries and checksums are the unchanged exact archives from main CI38128738119, source b68afc8. Full latest-head CI remains required before merge.

@antono3 antono3 left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Agent code review of 8ff642e.

Reviewed the consumer workflow permission change and matching release instructions. GitHub requires contents: write for draft visibility; these jobs only download assets and upload logs. Both draft-download consumers must pass. The exact release archives and their source commit are unchanged. No blocking findings.

All 8 applicable PR checks and complete latest-head workflows passed. This is agent review, not independent human approval.

@antono3
antono3 marked this pull request as ready for review October 11, 2026 10:19
@antono3
antono3 merged commit 3775d1b into main Oct 11, 2026
10 checks passed
@antono3
antono3 deleted the codex/draft-release-consumer-access branch October 11, 2026 10:19
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.

2 participants