ci: make Full Release the headline "📦 Release", disable the old one - #5608
ci: make Full Release the headline "📦 Release", disable the old one#5608adrians5j wants to merge 1 commit into
Conversation
|
🚓 Slop Cop ✅ Nothing worth flagging. The diff looks consistent with the PR's stated intent and the code-style rules. PR is coherent with its stated intent: renames the release-starting workflow, deletes the now-unused release.yml (generated artifact), and disables its source by renaming with a documented header. No integrity or style issues found. Automated, non-blocking heads-up from an LLM. It can be wrong — use your judgment. Regenerates on every push. |
`🚀 Full Release` is what you actually start a release with - it cuts the release branch and opens the PR - so it takes the plain name: `📦 Release`. That name belonged to a workflow that published to NPM by hand: `yarn release --type=beta --tag=<dist-tag>` followed by `--type=latest`, from a chosen branch. The /beta slash command runs that same beta-then-latest pair, but versioned and driven from a release PR, so the manual variant is unused. Its `.yml` is deleted - it no longer appears in the Actions list, and cannot be dispatched. The source is kept rather than deleted, renamed to `release.wac.disabled.ts`. `ghawac` globs `**/*.wac.ts`, so that suffix keeps it out of the build (the build now emits 14 files instead of 15) while leaving the steps readable if it ever needs to come back - rename it to `release.wac.ts` and rebuild. The frozen v5 workflows use the same trick with `.wac.frozen.ts`, the difference being that those still have their generated `.yml` checked in and this one does not. A header on the file records all of that. Nothing referenced the removed workflow: checked for `gh workflow run` callers, `workflow_run` triggers (there are none in the repo) and docs. Every remaining workflow name is unique. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
c2f29c8 to
5750481
Compare
|
Closing until the release queue is clear — reopen or re-cut when ready. Three release PRs are open right now (#5571 6.4.9, #5567 6.4.8, #5520 6.6.0). The rename half of this is harmless at any time, but deleting the old What this PR did, for whoever picks it up:
Also note: this branch was built on a stale |
What changed
🚀 Full Releaseis the workflow you actually start a release with — it cuts the release branch and opens the PR — so it takes the plain name:fullRelease.yml🚀 Full Release📦 Releaserelease.yml📦 ReleaseWhy the old one goes
It published to NPM by hand:
yarn release --type=beta --tag=<dist-tag>followed by--type=latest, from a chosen branch.The
/betaslash command runs that same beta-then-latest pair, but versioned and driven from a release PR — so the manual, branch-driven variant is unused. Its.ymlis deleted, so it no longer appears in the Actions list and cannot be dispatched.Disabled, not deleted
The source is kept as
release.wac.disabled.ts.ghawacglobs**/*.wac.ts, so that suffix keeps it out of the build — which now emits 14 files instead of 15 — while leaving the steps readable if it ever needs to come back:The frozen v5 workflows use the same trick with
.wac.frozen.ts; the difference is that those still have their generated.ymlchecked in and this one does not. A header on the file records what it did, why it is off, and how to restore it.Verification
gh workflow runcallers,workflow_runtriggers (none exist in the repo) and docs.oxfmt --checkandoxlintclean.Run history for the renamed workflow is preserved — a workflow's identity in the Actions UI is its file path, not its
name.Changelog
Clearer release workflows
The workflow that starts a release is now simply "Release". An unused manual NPM-publishing workflow has been removed from the Actions list.
Squash Merge Commit