Skip to content

fix: run real release without dry-run input - #7

Merged
f0rr0 merged 1 commit into
mainfrom
f0rr0/fix-release-publish-mode
Apr 26, 2026
Merged

fix: run real release without dry-run input#7
f0rr0 merged 1 commit into
mainfrom
f0rr0/fix-release-publish-mode

Conversation

@f0rr0

@f0rr0 f0rr0 commented Apr 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • split release-plz dry-run and publish into separate workflow steps
  • omit the dry_run input entirely on the real publish path
  • keep the no-op guard only for real publish runs

Verification

  • actionlint .github/workflows/release.yml .github/workflows/ci.yml .github/workflows/conventional-commits.yml
  • taplo fmt --check release-plz.toml Cargo.toml
  • git diff --check

Context

The failed publish run passed dry_run: false, but release-plz/action v0.5 treated the non-empty string false as dry-run and logged: due to dry, skipping the following: ["cargo registry upload", "creation of tag '0.2.0'", "creation of git release"]. This change makes publish omit that input entirely.

@f0rr0
f0rr0 merged commit bbf1ffc into main Apr 26, 2026
4 checks passed
@f0rr0
f0rr0 deleted the f0rr0/fix-release-publish-mode branch April 26, 2026 09:52
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