Prevent redundant release-plz PRs by separating release and release-pr triggers#67
Merged
Merged
Conversation
Agent-Logs-Url: https://github.com/wheregmis/dioxus-motion/sessions/4334eccf-864f-4eec-8cc7-7e541688533a Co-authored-by: wheregmis <26774729+wheregmis@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix release-plz workflow to prevent redundant release PRs
Prevent redundant release-plz PRs by separating May 22, 2026
release and release-pr triggers
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
release-plzwas running bothreleaseandrelease-pron every push tomain, which allowed a publish/tag to happen while still creating a follow-up reconciliation PR. This change separates those paths so publish does not race with PR generation.Workflow trigger separation
workflow_dispatchto.github/workflows/release-plz.yml.pushtomainfor automated release PR updates.Job-level gating
release-plz-releasenow runs only for manual dispatch:release-plz-prnow runs only for pushes:Resulting release flow
main→ create/update release PR only.This keeps release-plz behavior deterministic and avoids stale/duplicate release PR creation after a version is already released.
Original prompt
Fix the release-plz workflow in
wheregmis/dioxus-motionso it no longer creates redundant release PRs after a version has already been released.Context:
chore(dioxus-motion): release v0.3.5) was created by release-plz even thoughdioxus-motion-v0.3.5had already been released and tagged on April 7, 2026.release-plz releaseandrelease-plz release-pron every push tomainin.github/workflows/release-plz.yml.Relevant files already identified:
.github/workflows/release-plz.ymlrelease-plz.tomlCHANGELOG.mdPlease:
Notes:
command: releaseand another runningcommand: release-pr, both triggered bypushtomain.The following is the prior conversation context from the user's chat exploration (may be truncated):
User: even if its already released, release me is creating this PR. Identify the issue and lets fix this
Assistant: The issue is that your workflow always runs
release-prafterreleaseon every push tomain, so after publishingv0.3.5, it immediately opens a PR to add the changelog/version commit that the release step itself did not merge back intomain.Concretely:
Cargo.tomlonmainis already atversion = "0.3.5".dioxus-motion-v0.3.5was published on 2026-04-07.CHANGELOG.md, adding another0.3.5entry.main: one withcommand: release, and another withcommand: release-pr. That means the second job can still see “release metadata/changelog drift” and create a PR even after the release was already published.The likely root cause is: the release happened without the release PR being merged first, so the tag/release exists, but
mainnever got the corresponding changelog update. Thenrelease-prkeeps trying to reconcile that drift.Evidence
release-plz.ymlruns both jobs on every push tomain:PR #66 only modifies
CHANGELOG.mdand adds a duplicate0.3.5section:And the PR patch shows it is tryin...
This pull request was created from Copilot chat.