Skip to content

Conversation

extragloves
Copy link

@extragloves extragloves commented Sep 17, 2025

Removed i386 in example config.yaml since it's deprecated and the addon will not work

Proposed change

Type of change

  • Document existing features within Home Assistant
  • Document new or changing features for which there is an existing pull request elsewhere
  • Spelling or grammatical corrections, or rewording for improved clarity
  • Changes to the backend of this documentation
  • Remove stale or deprecated documentation

Checklist

  • I have read and followed the documentation guidelines.
  • I have verified that my changes render correctly in the documentation.

Additional information

  • This PR fixes or closes issue: fixes #
  • Link to relevant existing code or pull request:

Summary by CodeRabbit

  • Documentation
    • Updated Add-ons Tutorial: removed i386 from the supported architectures list in Step 1 (The basics), reflecting current compatibility (aarch64, amd64, armhf, armv7).
    • Clarifies platform support and reduces confusion for users on unsupported 32-bit x86.
    • No functional or runtime impact; all other tutorial content remains unchanged.

Removed i386 in example config.yaml since it's deprecated and the addon will not work
Copy link
Contributor

coderabbitai bot commented Sep 17, 2025

📝 Walkthrough

Walkthrough

Removed the i386 architecture from the list in the “Step 1: The basics” section of docs/add-ons/tutorial.md. No other content was modified.

Changes

Cohort / File(s) Summary
Docs update
docs/add-ons/tutorial.md
Updated architecture list by removing i386; list now shows aarch64, amd64, armhf, armv7. No other sections changed.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title Check ❓ Inconclusive The title "Update tutorial.md" names the file changed but is overly generic and does not communicate the primary change (removal of the deprecated i386 architecture from the example). As written the title will not tell reviewers or future readers the intent of the change at a glance. Because it is related to the changeset but non-descriptive, the check is inconclusive. Please rename the PR to a concise, descriptive title such as "Remove deprecated i386 from add-on tutorial architectures" or "Docs: remove i386 from docs/add-ons/tutorial.md example" so the primary change is clear. Updating the title will make the PR intent obvious to reviewers and changelog readers. If you prefer a different phrasing, ensure it mentions the removal of i386 and that it's a documentation change.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Description Check ✅ Passed The PR includes a clear top-line summary ("Removed i386 in example config.yaml since it's deprecated...") and correctly marks the change as "Remove stale or deprecated documentation," so the intent and scope are clear for this small docs-only change; however the repository template's "Proposed change" body is left blank, the "I have verified that my changes render correctly" checkbox is unchecked, and the Additional information section contains a placeholder "fixes #" with no issue number or links to the edited file (docs/add-ons/tutorial.md).
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

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