Skip to content

ci: move the release workflow to homebridge/.github - #38

Merged
NorthernMan54 merged 3 commits into
latestfrom
fix/release-fail-closed
Nov 2, 2026
Merged

NorthernMan54 merged 3 commits into
latestfrom
fix/release-fail-closed

Conversation

@NorthernMan54

@NorthernMan54 NorthernMan54 commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

Why

This run, started manually on latest, reached npm publish --tag none. It only failed because 2.2.6 was already published.

What

release.yml now holds only its triggers, permissions and concurrency, and calls the shared npm-release.yml, which fails on anything other than a release or a beta-*/alpha-* branch.

Merge order

Merge after homebridge/.github#59. This calls the shared workflow at @latest, so it breaks until that PR is in.

Notes

  • cancel-in-progress is now false, so a new push can't cancel a run after npm publish and leave a version with no pre-release.
  • The npm trusted publisher filename stays release.yml, so no npm change should be needed.
  • Merging doesn't publish. A beta branch must be cut from latest after the merge to pick this up.

Testing

Not run on GitHub Actions yet. After both PRs merge, a manual run on latest should fail at the first job with nothing published. The next beta-* push is the first real test of publishing through the shared workflow, so check that one closely.

🤖 Generated with Claude Code

NorthernMan54 and others added 2 commits November 2, 2026 08:58
Use the shared determine-npm-release workflow, which fails on any event and
branch it does not recognise, so publish is skipped.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
release.yml now only holds the triggers, permissions and concurrency, and
calls the shared npm-release workflow.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@NorthernMan54 NorthernMan54 changed the title fix(ci): stop manual release runs from publishing with the none tag ci: move the release workflow to homebridge/.github Nov 2, 2026
A cancelled run could leave a version on npm with no pre-release. New pushes
now wait for the running release to finish.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@NorthernMan54
NorthernMan54 merged commit 9779704 into latest Nov 2, 2026
7 checks passed
@NorthernMan54
NorthernMan54 deleted the fix/release-fail-closed branch November 2, 2026 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant