Skip to content

Add release-plz to use trusted publishing#3027

Open
Kobzol wants to merge 1 commit into
rust-lang:mainfrom
Kobzol:main
Open

Add release-plz to use trusted publishing#3027
Kobzol wants to merge 1 commit into
rust-lang:mainfrom
Kobzol:main

Conversation

@Kobzol

@Kobzol Kobzol commented Jul 18, 2026

Copy link
Copy Markdown
Member

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 18, 2026

@taiki-e taiki-e 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.

Thanks for the PR! However, I suspect release-plz might not work well with futures-rs's release method (which is based on backports using cherry-pick. e.g., #3024)

Also, this repository is not currently creating releases from the main branch.

View changes since this review

Comment on lines +26 to +28
- &install-rust
name: Install Rust toolchain
uses: dtolnay/rust-toolchain@stable

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.

As ci.yml does, please use taiki-e/install-action instead of these actions. (needed to pass zizmor check in ci/tidy job)

- name: Install Rust
uses: taiki-e/install-action@c93ccc03e00cd0e08e494f5fd058a6c55a6a1907 # v2.82.8
with:
tool: rust@nightly
fallback: none

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 18, 2026
# Release unpublished packages.
release-plz-release:
name: Release-plz release
if: ${{ github.repository_owner == 'rust-lang' }}

@taiki-e taiki-e Jul 18, 2026

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.

${{ }} is needless in if: .

Suggested change
if: ${{ github.repository_owner == 'rust-lang' }}
if: github.repository_owner == 'rust-lang'

View changes since the review

# Create a PR with the new versions and changelog, preparing the next release.
release-plz-pr:
name: Release-plz PR
if: ${{ github.repository_owner == 'rust-lang' }}

@taiki-e taiki-e Jul 18, 2026

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.

Suggested change
if: ${{ github.repository_owner == 'rust-lang' }}
if: github.repository_owner == 'rust-lang'

View changes since the review

@Kobzol

Kobzol commented Jul 18, 2026

Copy link
Copy Markdown
Member Author

Sure, happy to change the release method! We can release based on git tags, or GitHub releases, for example.

Regarding releasing from main, that's how release-plz works in this configuration, it runs on main to observe if a release PR was merged, and then performs the release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants