-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Add Rehlko Kohler 30RCL to list of generators that work. #39735
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
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📝 WalkthroughWalkthroughThe documentation for the Rehlko integration was updated to add the Kohler 30RCL generator model to the list of tested generators, including a link to its product page. No code, logic, or control flow changes were made. Changes
Sequence Diagram(s)(No sequence diagram generated as the changes are documentation-only.) Assessment against linked issues
Assessment against linked issues: Out-of-scope changes(No out-of-scope changes found.) 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:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. 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 (1)
source/_integrations/rehlko.markdown (1)
101-109
: Optional: keep the “Tested generators” list ordered for quick scanningThe bullet list is now
38 → 26 → 20 → 20 → 14 → 30
, which feels random. Consider ordering numerically (descending or ascending) or alphabetically for easier maintenance.Purely cosmetic—no functional impact.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Cache: Disabled due to data retention organization setting
Knowledge Base: Disabled due to data retention organization setting
📒 Files selected for processing (1)
source/_integrations/rehlko.markdown
(1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
`source/_integrations/*`: Integration documentation must use the provided template structure and section order.
source/_integrations/*
: Integration documentation must use the provided template structure and section order.
📄 Source: CodeRabbit Inference Engine (.github/copilot-instructions.md)
List of files the instruction was applied to:
source/_integrations/rehlko.markdown
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: Redirect rules - home-assistant-docs
- GitHub Check: Header rules - home-assistant-docs
- GitHub Check: Pages changed - home-assistant-docs
🔇 Additional comments (1)
source/_integrations/rehlko.markdown (1)
108-108
: Model/link mismatch – please verify generator identifierThe list item shows
30RCL
, but the linked URL ends with30rcla
. That extra “A” denotes a different model (30RCLA
). Either the text or the link (or both) needs to be corrected to avoid confusing readers.- - [30RCL](https://www.kohlerhomeenergy.rehlko.com/products/home+generators/30rcla) + - [30RCLA](https://www.kohlerhomeenergy.rehlko.com/products/home+generators/30rcla)If you truly mean the non-“A” variant, update the URL to the exact 30RCL page instead.
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.
Pull Request Overview
This PR adds the Kohler 30RCL model to the list of generators tested with the Rehlko integration.
- Included the 30RCL series in the tested generators section.
Comments suppressed due to low confidence (1)
source/_integrations/rehlko.markdown:108
- [nitpick] The display name
30RCL
does not match the URL suffix30rcla
. Consider updating the link text to30RCL-A
(if the model includes the ‘A’ suffix) or removing the trailing ‘a’ in the URL to keep them consistent.
- [30RCL](https://www.kohlerhomeenergy.rehlko.com/products/home+generators/30rcla)
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, @PeteRager 👍
../Frenck
Proposed change
Add the 30RCL series to generators that work with the integration.
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.next
branch.Summary by CodeRabbit