Skip to content

Rest refactor - #5

Merged
nolsen311 merged 4 commits into
mainfrom
REST-refactor
Sep 6, 2026
Merged

nolsen311 merged 4 commits into
mainfrom
REST-refactor

Conversation

@nolsen311

Copy link
Copy Markdown
Owner

No description provided.

nolsen311 and others added 3 commits September 6, 2026 10:08
On pull_request events actions/checkout lands on refs/pull/N/merge, a
detached HEAD, so stefanzweifel/git-auto-commit-action warns and has to
guess the branch from GITHUB_HEAD_REF before it can push the ruff fixes.
Pin `ref` to github.head_ref (pull_request) or github.ref_name (push)
so the checkout is on a real branch.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
main is on the 0.9.x release line; this branch still carried 3.0.0 from
before the first releases existed. Merging that would push main's
manifest above the release line and leave HACS unable to offer any
0.x release as an update. Align it with the next planned tag.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
release.yml used to rewrite manifest.json from the tag name *after* the
release was published, so every tag shipped the previous version's
manifest (v0.9.1 installs report 0.9.0). Instead:

- manifest.json is bumped in-repo (here, to 0.9.3) so the tagged commit
  already carries the right version.
- release.yml no longer edits or commits the manifest. It now just
  verifies the tag matches manifest.json (failing the release if not),
  builds the zip, and uploads it.

Supersedes the interim 0.10.0 bump on this branch.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown

🎉 Thank you for contributing to SkyRadar Fusion! Your Pull Request has been received. Our automated tests (Hassfest, Ruff, etc.) will run shortly. If anything fails, don't worry—just check the logs and update your branch!

@nolsen311
nolsen311 merged commit d397ef4 into main Sep 6, 2026
5 of 10 checks passed
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