-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Fixes for 2025-04 release notes #38320
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
It seems that this PR is targeted against an incorrect branch. Documentation updates which apply to our current stable release should target the |
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
📝 WalkthroughWalkthroughThe release notes document has been updated to improve clarity and detail. The section on dashboard features has been renamed from "Introducing the areas dashboard (experimental)" to "Smarter dashboards!" and now includes additional information on the automatic generation of dashboards based on user-defined areas. Similarly, the onboarding section has been renamed from "Restoring backup from Home Assistant Cloud during onboarding" to "Onboarding with a Home Assistant Cloud backup" with expanded details on backup restoration. Minor grammatical corrections have also been applied throughout. Changes
Sequence Diagram(s)sequenceDiagram
participant U as User
participant A as Areas Setup
participant D as Dashboard Generator
U->>A: Define areas and preferences
A->>D: Trigger auto-generation of dashboard
D->>U: Display customized dashboard
sequenceDiagram
participant U as User
participant O as Onboarding Process
participant B as Backup Manager
U->>O: Initiate onboarding
O->>B: Request Home Assistant Cloud backup
B-->>O: Provide backup data
O->>U: Complete onboarding with restored settings
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (13)
source/_posts/2025-04-02-release-20254.markdown (13)
79-81
: Navigation Updates – Verify Updated Links
The table of contents has been updated to reflect the new section titles and hierarchy (“Smarter dashboards!” and its sub-item “A new experimental Areas dashboard”). Please verify that the fragment identifiers (#smarter-dashboards
and#a-new-experimental-areas-dashboard
) correctly match the respective section headers.
152-156
: Dashboard Section Heading & Intro Tone
The revised heading “## Smarter dashboards!” and introductory copy are clearer and more engaging. A couple of notes:
- The trailing exclamation mark may trigger markdownlint MD026; consider if it’s acceptable per style guidelines or if the exclamation should be removed for consistency.
- The informal term “automagically” adds a playful tone; please confirm this aligns with the intended style for the project’s documentation.
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
152-152: Trailing punctuation in heading
Punctuation: '!'(MD026, no-trailing-punctuation)
163-163
: Image Alt Text Placeholder
The image on this line uses the alt text “TODO: Replace.” For accessibility and professionalism, please update the alt text to a descriptive caption before the final release.
169-170
: Overview Page Call-to-Action Clarity
The explanation of the Overview page is clear. As a minor nitpick, consider using an em dash (—) or a colon instead of a simple hyphen in the sentence “...into areas - it’s the key to unlocking this new experience…” to improve the flow and punctuation consistency.
263-263
: Pre-announce Sound – Tone Review
The sentence “Having your assistant start a conversation is great, but having it start talking out of nowhere might be a bit scary for some people 🫣.” employs a friendly, informal tone. Please confirm that this style aligns with the overall documentation standards; otherwise, consider a slightly more formal phrasing.
265-265
: Jump Scare Explanation – Informal Phrase Caution
“Using this, you can play a little sound before the assistant starts talking, preventing the ‘jump scare’ effect.” is clear. However, the phrase “jump scare” may be too informal for some documentation standards. Verify that it’s acceptable or consider a more neutral term.
276-278
: Cloud Backup Description Refinement
The revised copy explaining the Home Assistant Cloud backup service is more concise and informative. You might want to consider a minor rephrase for enhanced flow (for example, breaking up long sentences) to further improve readability.
280-280
: Sentence Structure Improvement Needed
In the sentence “are migrating to a new Home Assistant Green, as of this release, you can directly restore your backup…”, the comma usage creates a run-on feel. Splitting this into two sentences or using a semicolon would improve clarity.🧰 Tools
🪛 LanguageTool
[style] ~280-~280: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...id happen, maybe a hardware failure, or maybe you are migrating to a new [Home Assist...(REP_MAYBE)
285-285
: Cloud Backup Image Alt Text Update
The image on this line uses alt text “TODO: Replace.” Please update this placeholder with a descriptive caption that explains the image content for both accessibility and clarity.
403-403
: Additional Card Interactions – Tone and Punctuation Review
The new feature announcement “We now have support for adding additional interactions to cards! Hold and double tap actions are now available through the UI. Thanks [@piitaya]!” is engaging. Consider using a more neutral punctuation (e.g., a period instead of an exclamation mark) for a professional tone, and double-check that the phrasing “Hold and double tap actions” is as clear as intended.
259-259
: Duplicate Link Reference – [pve]
Static analysis reports a duplicate link reference definition for[pve]
. Please confirm whether there are two definitions present; if so, remove the redundant one to comply with markdown guidelines (MD053).🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
259-259: Link and image reference definitions should be needed
Duplicate link or image reference definition: "pve"(MD053, link-image-reference-definitions)
242-242
: Wording Consistency – Substitute “amazing”
A static analysis hint suggests reconsidering the use of “amazing” (as seen in a context related to the Voice Preview Edition). To maintain a consistent and more formal tone, consider substituting “amazing” with alternatives like “impressive” or “exceptional.”🧰 Tools
🪛 LanguageTool
[style] ~242-~242: Consider using a more formal and expressive alternative to ‘amazing’.
Context: ...t Voice Preview Edition][pve]! This is an amazing feature, as it allows you to build your...(AWESOME)
252-253
: Variety in Language – Repetitive Phrasing
There is a repeated idiomatic phrasing in this section. For example, “in the mood for some music” might be unnecessarily wordy, and similar expressions appear nearby. Consider revising these sections for language variety and conciseness.🧰 Tools
🪛 LanguageTool
[style] ~252-~252: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ... a great day! Want to hear the news? Or maybe you are in the mood for some music?" an...(REP_MAYBE)
[style] ~253-~253: ‘in the mood for’ might be wordy. Consider a shorter alternative.
Context: ...Want to hear the news? Or maybe you are in the mood for some music?" and answering that? This ...(EN_WORDINESS_PREMIUM_IN_THE_MOOD_FOR)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
source/images/blog/2025-04/areas-experimental-dialog.png
is excluded by!**/*.png
📒 Files selected for processing (1)
source/_posts/2025-04-02-release-20254.markdown
(12 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
source/_posts/2025-04-02-release-20254.markdown
152-152: Trailing punctuation in heading
Punctuation: '!'
(MD026, no-trailing-punctuation)
259-259: Link and image reference definitions should be needed
Duplicate link or image reference definition: "pve"
(MD053, link-image-reference-definitions)
🪛 LanguageTool
source/_posts/2025-04-02-release-20254.markdown
[style] ~242-~242: Consider using a more formal and expressive alternative to ‘amazing’.
Context: ...t Voice Preview Edition][pve]! This is an amazing feature, as it allows you to build your...
(AWESOME)
[style] ~252-~252: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ... a great day! Want to hear the news? Or maybe you are in the mood for some music?" an...
(REP_MAYBE)
[style] ~253-~253: ‘in the mood for’ might be wordy. Consider a shorter alternative.
Context: ...Want to hear the news? Or maybe you are in the mood for some music?" and answering that? This ...
(EN_WORDINESS_PREMIUM_IN_THE_MOOD_FOR)
[style] ~280-~280: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...id happen, maybe a hardware failure, or maybe you are migrating to a new [Home Assist...
(REP_MAYBE)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: Lint Markdown
🔇 Additional comments (9)
source/_posts/2025-04-02-release-20254.markdown (9)
157-162
: New Experimental Areas Dashboard Subsection
The new subsection title “### A new experimental Areas dashboard” and its accompanying description provide a concise overview of the feature. The content is informative and well structured.
186-186
: Time Format Support Statement
The sentence “It supports a few time formats, displaying different timezones, and of course, can be resized to match your needs.” reads clearly. No further changes are required here.
216-216
: Enhanced Voice Wizard Description
The updated wording—“The wizard will now help you make a more informed decision based on your language, desired functionality, and device capabilities…”—is clear and helpful. This improvement should aid users in understanding the enhanced voice setup experience.
222-222
: Conversational Tone – Voice Assistant Interaction
The revision “Ever tried to have a conversation with your voice assistant that is hooked up to an LLM like ChatGPT?” reads naturally and invites engagement.
225-225
: LLM Conversation Continuity Explanation
The sentence “If the LLM returns with a question, we will detect that and keep the conversation going, without the need for you to say ‘Ok Nabu’ again.” is clear and informative.
231-231
: LLM Support Confirmation
The short statement “assistants, and it works with all LLMs supported by Home Assistant.” confirms compatibility well.
273-273
: Onboarding Section Title Update
The updated title “## Onboarding with a Home Assistant Cloud backup” improves clarity and focus regarding the backup restoration process.
427-427
: Template Functions Overview Clarity
The introduction to the new template functions (“More noteworthy is the addition of a series of new template functions to make working with data a lot easier:”) is clear and helpful. No changes needed here.
456-456
: Device Hierarchy Explanation
The explanation regarding the device hierarchy (“This is great if, for example, you have a breaker that is monitoring the energy consumption of a whole circuit, but also have individual devices on that circuit that are also monitored.”) is informative. A very minor punctuation tweak could improve flow, but overall this paragraph works well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @googanhiem ❤️
Proposed change
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.next
branch.Summary by CodeRabbit