Skip to content

Defer release publication until publish#392

Merged
yuecideng merged 1 commit into
mainfrom
fix/draft-release-workflows
Jul 13, 2026
Merged

Defer release publication until publish#392
yuecideng merged 1 commit into
mainfrom
fix/draft-release-workflows

Conversation

@yuecideng

Copy link
Copy Markdown
Contributor

Description

This PR prevents a GitHub release draft tag from publishing EmbodiChain distributions to PyPI or deploying versioned documentation.

Tag pushes continue to build CI artifacts. PyPI publication and release-doc deployment now run only when the associated GitHub release is published.

Dependencies: None.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (non-breaking change which improves an existing functionality)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (existing functionality will not work without user modification)
  • Documentation update

Screenshots

Not applicable.

Checklist

  • I have run the Black formatting check.
  • I have made corresponding changes to the documentation
  • I have run the relevant docs publication tests.
  • Dependencies have been updated, if applicable.

Copilot AI review requested due to automatic review settings July 13, 2026 16:54
@yuecideng yuecideng added bug Something isn't working ci labels Jul 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 changes the release pipeline so that building artifacts still happens on tag pushes, but publishing (PyPI + versioned docs deployment) is deferred until the corresponding GitHub Release is published.

Changes:

  • Added a dedicated release-triggered workflow to publish to PyPI only when a release is published (and tag starts with v).
  • Removed the tag-push-driven PyPI publish job from the main CI workflow.
  • Updated the docs Pages workflow to deploy versioned docs on release.published (and restrict artifact-based deploys to successful main pushes).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/release-publish.yml New workflow that builds from the published tag and publishes to PyPI via Trusted Publishing.
.github/workflows/main.yml Removes the previous tag-push PyPI publishing job while keeping tag builds/artifacts.
.github/workflows/docs-pages.yml Adds release.published as a deployment trigger and tightens artifact deploy conditions to main pushes.

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

Comment on lines +22 to +25
- uses: actions/checkout@v4
with:
ref: ${{ github.event.release.tag_name }}

@yuecideng yuecideng merged commit 95fa94f into main Jul 13, 2026
5 checks passed
@yuecideng yuecideng deleted the fix/draft-release-workflows branch July 13, 2026 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ci

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants