Skip to content

Polish hottubctl for public use - #1

Merged
cnberry merged 2 commits into
mainfrom
codex/unified-tool-style
Aug 16, 2026
Merged

Polish hottubctl for public use#1
cnberry merged 2 commits into
mainfrom
codex/unified-tool-style

Conversation

@cnberry

@cnberry cnberry commented Aug 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • add matching hero art, README structure, operations/protocol/troubleshooting/roadmap docs, MIT license, security policy, and CI
  • move passwords out of the public config example with HOTTUBCTL_PASSWORD support
  • require --yes for temperature writes, validate units, and close failed login sessions
  • align with switchctl and the shared language-neutral script/install contract
  • declare the Python 3.13+ requirement imposed by python-smarttub 0.0.47

Verification

  • 7 unit tests
  • Ruff format and lint
  • detect-secrets baseline scan
  • installer syntax and executable checks
  • wheel build, CLI help, Markdown link, and current-tree privacy checks

@cnberry
cnberry marked this pull request as ready for review August 16, 2026 21:26
@cnberry
cnberry merged commit 7538def into main Aug 16, 2026
4 checks passed
@cnberry
cnberry deleted the codex/unified-tool-style branch August 16, 2026 21:26
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