Skip to content

fix(gha): flip merged release PR label to autorelease: tagged#302

Open
so0k wants to merge 2 commits into
mainfrom
worktree-release-please-autorelease-tagged
Open

fix(gha): flip merged release PR label to autorelease: tagged#302
so0k wants to merge 2 commits into
mainfrom
worktree-release-please-autorelease-tagged

Conversation

@so0k

@so0k so0k commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • release-please-config.json sets skip-github-release: true so release.yml's stable job (not release-please) creates the tag/GitHub Release, avoiding a double-create race.
  • Because release-please never creates that release itself, it never flips the merged release PR's label from autorelease: pending to autorelease: tagged.
  • Release-please refuses to open a new release PR while a merged one is still labeled autorelease: pending ("There are untagged, merged release PRs outstanding - aborting") — this silently blocked every release PR after v0.23.4 shipped.
  • Adds a step to the release_github job that relabels the merged PR right after the GitHub release is created, so release-please's bookkeeping matches reality going forward.

PR #254 (v0.23.4) was manually relabeled autorelease: tagged already to unblock the standing PR immediately; this change prevents the same thing from recurring on every future release.

One-off setup performed by hand (label didn't exist yet)

gh label create "autorelease: tagged" --repo open-constructs/cdk-terrain --color 0E8A16 \
  --description "This PR represents a release that has been tagged and published"

gh pr edit 254 --repo open-constructs/cdk-terrain \
  --remove-label "autorelease: pending" --add-label "autorelease: tagged"

Test plan

  • actionlint .github/workflows/release.yml — no new findings (only pre-existing unrelated warnings)
  • Next stable release run creates a GitHub Release and the relabel step succeeds
  • Release Please workflow opens a new release PR afterward instead of aborting

release-please-config.json sets skip-github-release: true so the
release.yml stable job (not release-please) creates the tag/GitHub
Release. Release-please never sees that as "finalized" and leaves the
merged release PR labeled autorelease: pending forever, which makes it
permanently refuse to open the next release PR ("There are untagged,
merged release PRs outstanding - aborting").

Have the release_github job relabel the merged PR to
autorelease: tagged right after it creates the GitHub release, so
release-please's own bookkeeping matches reality.
@so0k so0k requested a review from a team as a code owner July 4, 2026 09:27
Comment thread .github/workflows/release.yml Outdated
The label is a one-time, repo-wide fixture (already created), so
attempting to create it on every release was unnecessary.

Addressed feedback: https://github.com/open-constructs/cdk-terrain/pull/302/changes#r3522958993
@so0k so0k added ci/skip-examples Skip Examples Testing in PR ci/skip-integration Skips Integration Testing on PR ci/skip-provider-integration Skips Provider Integration Tests on PR ci/skip-unit Skips Unit Testing on PR labels Jul 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/skip-examples Skip Examples Testing in PR ci/skip-integration Skips Integration Testing on PR ci/skip-provider-integration Skips Provider Integration Tests on PR ci/skip-unit Skips Unit Testing on PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant