ci(github): auto-fire release after merge of 'PR: Release after merge' labeled PRs - #37384
ci(github): auto-fire release after merge of 'PR: Release after merge' labeled PRs#37384wezell wants to merge 1 commit into
Conversation
❌ Linked Issue Needs Team LabelThis PR is linked to issue #37383, but that issue has no How to fix this:Apply a This comment was automatically generated by the issue linking workflow |
|
Claude finished @wezell's task in 1m 0s —— View job Code ReviewI reviewed the new workflow New Issues
Notes (non-blocking)
Once the casing bug is fixed, this looks good to merge. Note: I can't modify files under |
Fixes #37383
Summary
Adds a new workflow,
issue_release-after-merge.yml, that automatically fires the dotCMS release process when a PR labeledPR: Release after mergeis merged intomain.How it works
pull_request: closedwithmerged == true, base refmain, and thePR: Release after mergelabel present at merge time.v{yy.mm.dd}-##(zero-padded two-digit counter, UTC date):v{yy.mm.dd}-##tags viagit ls-remotecicd_6-release.ymlruns queued or in flight, parsing their versions from run names (Release <version>), and steps over them (their tags don't exist yet)max(tag counter, in-flight counter) + 1, re-verifying each candidate against the remote — important becauserelease-prepareforce-overwrites colliding tags/branchesgh workflow run cicd_6-release.yml --ref mainwith the computedrelease_version.Design decisions
pull_request: closed, not the merge queueissue_post-pr-merge.ymlauto-release-after-merge)CI_MACHINE_TOKENGITHUB_TOKENdo not trigger other workflow runs (recursive-workflow prevention) — agithub.tokendispatch would silently no-opbase.ref == 'main'verify-branchfail-fastSDK Breaking Change→bump_min_sdk_version=trueTesting
04, in-flight max07→ correctly computed08)Notes
PR: Release after mergelabel to exist on the reposecrets.CI_MACHINE_TOKEN(already used by the release workflow itself)