Skip to content

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

Merged
so0k merged 2 commits into
mainfrom
worktree-release-please-autorelease-tagged
Jul 5, 2026
Merged

fix(gha): flip merged release PR label to autorelease: tagged#302
so0k merged 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
@so0k
so0k merged commit 8d64f6c into main Jul 5, 2026
207 of 275 checks passed
@so0k
so0k deleted the worktree-release-please-autorelease-tagged branch July 5, 2026 12:24
@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

2 participants