Skip to content

Conversation

@frenck
Copy link
Member

@frenck frenck commented Nov 12, 2025

Proposed Changes

The Home Assistant project has deprecated armv7 support, pending for removal in the Home Assistant 2025.12 release. This PR removes it from this add-on.

../Frenck

                       

Blogging my personal ramblings at frenck.dev

Summary by CodeRabbit

  • Chores
    • Removed support for armhf, armv7, and i386 architectures. The application now supports aarch64 and amd64 only.

@frenck frenck added the breaking-change A breaking change for existing users. label Nov 12, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 12, 2025

Walkthrough

The PR removes support for three architectures (armhf, armv7, i386) across the project's build configuration, deployment settings, and documentation, narrowing supported platforms to aarch64 and amd64 only.

Changes

Cohort / File(s) Summary
CI and Documentation
.github/renovate.json, README.md
Narrowed Dockerfile regex to only match aarch64 and amd64 architectures for Renovate; removed architecture shields and their definitions for armhf, armv7, and i386 from README
Build Configuration
zwave-js-ui/build.yaml, zwave-js-ui/config.yaml
Removed armv7 from build matrix and architecture list, reducing supported architectures from three to two

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • The changes are straightforward removals of architecture entries across configuration files with a consistent pattern
  • Review focus should be on the regex modification in renovate.json to ensure the pattern correctly matches only the intended architectures (aarch64|amd64)

Suggested labels

bugfix

Poem

🐰 Four architectures once stood tall,
But now we trim them, less is all.
Armhf, armv7, i386—goodbye,
Aarch64 and amd64 now fly high!
Simpler builds, cleaner dreams—
Less complexity, it seems! 🏗️

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and accurately summarizes the main change: removing armv7 architecture support across all modified files (renovate.json, README.md, build.yaml, and config.yaml).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch frenck-2025-0478

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c480a51 and 5b45281.

📒 Files selected for processing (4)
  • .github/renovate.json (1 hunks)
  • README.md (0 hunks)
  • zwave-js-ui/build.yaml (0 hunks)
  • zwave-js-ui/config.yaml (0 hunks)
💤 Files with no reviewable changes (3)
  • zwave-js-ui/build.yaml
  • README.md
  • zwave-js-ui/config.yaml
🔇 Additional comments (1)
.github/renovate.json (1)

15-15: Correct architecture narrowing for Renovate dependency updates.

The regex pattern restriction to (aarch64|amd64) is properly aligned with the PR objective to drop armv7, armhf, and i386 support. Renovate will now only capture Docker base image updates for the remaining supported architectures, preventing unnecessary dependency management for removed platforms.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@frenck frenck merged commit 01175bc into main Nov 12, 2025
20 checks passed
@frenck frenck deleted the frenck-2025-0478 branch November 12, 2025 19:01
@github-actions github-actions bot locked and limited conversation to collaborators Nov 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

breaking-change A breaking change for existing users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants