Skip to content

Prepare Grott 0.1.10-beta release - #4

Merged
Herbertmt978 merged 4 commits into
masterfrom
Herb/wave0-stabilization
Jul 14, 2026
Merged

Prepare Grott 0.1.10-beta release#4
Herbertmt978 merged 4 commits into
masterfrom
Herb/wave0-stabilization

Conversation

@Herbertmt978

Copy link
Copy Markdown
Owner

Summary

Prepare the v0.1.10-beta Grott HA Docker release candidate for the fork.

This PR includes the Wave 0 stabilization and release hardening work, switches the release model to solo-maintainer controls, and records the fork owner confirmation that upstream redistribution permission has been obtained.

The permission note is conditional: it does not authorize commercial use or reuse unless Johan Meijer has separately agreed and any financial reward or appreciation is directed to him.

Validation

  • python tools/validate_release.py --check-worktree --tag v0.1.10-beta
  • python tools/validate_ha_addon_repo.py
  • python -m compileall -q grott.py grottconf.py grottdata.py grottlayout.py grottprotocol.py grottproxy.py grottserver.py grottsniffer.py grottext tools
  • python -m pytest tests "examples/Home Assistent/test_grott_ha.py" -q (476 passed locally)
  • python -m pip check

Hosted CI on this PR remains the required merge gate.

Release Guardrails

No tag or release is created by this PR. After merge, release still requires a successful master CI run at the exact merge SHA before creating the annotated v0.1.10-beta tag and manually dispatching publish-ghcr.yml.

@Herbertmt978
Herbertmt978 merged commit 4b293ba into master Jul 14, 2026
1 check passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 25f7859cd0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread addons/grott/Dockerfile
Comment on lines +62 to +66
COPY grott.py grottconf.py grottdata.py grottlayout.py grottprotocol.py grottproxy.py grottserver.py grottsniffer.py /app/
COPY grottext /app/grottext
COPY examples/grott.ini /app/grott.ini
COPY ["examples/Home Assistent/grott_ha.py", "/app/grott_ha.py"]
COPY T06NNNNXMOD.json /app/T06NNNNXMOD.json

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Restore bundled external layouts in the add-on image

For add-on users whose packets rely on layouts that are only shipped as external JSON files under examples/Record Layout, this reduced copy set drops those files from /app. grottconf.set_reclayouts() walks the working directory for T*.json/t*.json, and the previous add-on image cloned the whole repo into /app, so layouts such as t060103xmax3.json and T06221b.json were auto-loaded; this image now includes only T06NNNNXMOD.json, causing those record types to be treated as undefined unless users rebuild or mount missing layouts.

Useful? React with 👍 / 👎.

@Herbertmt978
Herbertmt978 deleted the Herb/wave0-stabilization branch July 15, 2026 08:22
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