Skip to content

ci(release): open the release PR only after the tag job - #60

Merged
StefanoGuerrini merged 1 commit into
mainfrom
ci/release-pr-after-tag
Oct 16, 2026
Merged

StefanoGuerrini merged 1 commit into
mainfrom
ci/release-pr-after-tag

Conversation

@zenrows-dev-pool

@zenrows-dev-pool zenrows-dev-pool Bot commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The release-PR job now runs after the tag job. On the push that merges a release PR, the two jobs ran side by side. When the release-PR job read the releases first, it found none for the new version and opened a release PR that listed changes already shipped.

What the task asked for

Stop release-please from proposing releases that repeat shipped changes.

How the check confirmed it

actionlint passes on release.yml. In one release run, the release-PR job logged Expected 1 releases, only found 0. The tag job created that release 39 seconds later. The release-PR job had already opened a release PR from the whole history.

Encoded by: none.

How to try it

Nothing to run before merge. After the next release PR merges, the run shows Open or update the release PR starting after Tag, release and publish.

Provenance

On a push that merges no release PR, GitHub skips the release job. needs.release.result is then skipped, and the release-PR job runs as before. A failed release job skips it, so a re-run handles both jobs. If build fails, GitHub skips release instead, and the release-PR job runs. It then finds the merged release PR still untagged and opens nothing.

Review

Severity File and line Summary
None — One pass, by the author. No findings.

Human must verify:

  • If the release environment needs approval, the release-PR job waits for that approval too.

Deviations

None.

Record owed

None.

On the push that merges a release PR, the release-PR job ran beside the
tag job. When it read the releases first, it found none for the new
version, read the whole history as unreleased, and opened a release PR
that listed changes already shipped. It now waits for the tag job.
@zenrows-dev-pool
zenrows-dev-pool Bot requested a review from a team as a code owner October 16, 2026 11:21
@StefanoGuerrini
StefanoGuerrini merged commit ceb4931 into main Oct 16, 2026
2 checks passed
@StefanoGuerrini
StefanoGuerrini deleted the ci/release-pr-after-tag branch October 16, 2026 11:45
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