feat(ci): update prior release workflow - #4854
hopehadfield wants to merge 2 commits into
Conversation
PR Summary by QodoSupport repository-wide prior releases for 2.x branches
AI Description
Diagram
High-Level Assessment
Files changed (4)
|
| registry-url: https://registry.npmjs.org/ # Needed for auth | ||
|
|
||
| - name: Install root dependencies | ||
| run: yarn install --immutable |
Code Review by Qodo
1. Unsupported 2.0 releases run
|
|
Important The |
Signed-off-by: Hope Hadfield <hhadfiel@redhat.com>
|
|
Lifecycle scripts must remain enabled in the publish job because the release build depends on native package bindings. There is no risk here because the job only runs for merged, bot-created Version Packages PRs. |
lokanandaprabhu
left a comment
There was a problem hiding this comment.
Looks mostly good to me — thanks for splitting the legacy 1.x release-1.x/{plugin} path from the 2.1+ repository-wide release-x.y flow.
One question on the old workspace/* path: this PR removes workspace/** from the workflow on main, but existing workspace/<plugin> branches still carry the old release_workspace_version.yml. GitHub runs that file from the target branch, so a manual backport merged into workspace/<plugin> can still open Version Packages even after this lands.
How do we want to handle that?
sync the updated workflow onto those branches so they no longer trigger Prior Version Release or delete them after confirming nothing is pending in those branches?




Hey, I just made a Pull Request!
Addresses https://redhat.atlassian.net/browse/RHIDP-16860
Updates the prior-release workflow to support the new repository-wide release-2.1+ model while preserving the legacy per-plugin release-1.9/1.10/<workspace> flow.
Changes
The duplication is intentional because the legacy 1.x and repository-wide 2.1+ workflows use different branch structures, workspace discovery, and release handling. Keeping the paths separate also avoids introducing unnecessary breakage through a broad refactor during the transition; the legacy path can be removed after 1.x reaches EOL.
Validation
Tested in my personal fork with no package publication or tag creation:
✔️ Checklist