Skip to content

ci: gate releases on shared verification - #1814

Merged
eitsupi merged 1 commit into
mainfrom
ci/verify-before-release
Oct 2, 2026
Merged

eitsupi merged 1 commit into
mainfrom
ci/verify-before-release

Conversation

@eitsupi

@eitsupi eitsupi commented Oct 2, 2026 •

Copy link
Copy Markdown
Member

Gate pre-releases and tagged releases on the existing build, lint, and test workflow.
Part of #1732

@eitsupi
eitsupi marked this pull request as ready for review October 2, 2026 13:48
@eitsupi
eitsupi requested review from Fred-Wu, ManuelHentschel, randy3k and renkun-ken and a lite review from Copilot October 2, 2026 13:51

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

Move pre-release concurrency to the job level to prevent eligible publications from being canceled prematurely.

Review effort: Lite
Findings: None

What changed in this PR

Gates tagged and development releases on the shared verification workflow.

Changes:

  • Makes main.yml reusable.
  • Gates releases on successful verification.
  • Uses verified artifacts for publishing.
  • Updates pre-release documentation.
File Summary
README.md Documents verification-gated development releases.
.github/​workflows/​release.yml Gates releases and publishing on verification.
.github/​workflows/​pre-release.yml Publishes verified main revisions.
.github/​workflows/​main.yml Enables reusable workflow invocation.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@ManuelHentschel ManuelHentschel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can't check locally, but the changes look sound to me.

@eitsupi
eitsupi merged commit b963208 into main Oct 2, 2026
6 checks passed
@eitsupi
eitsupi deleted the ci/verify-before-release branch October 2, 2026 23:20
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.

3 participants