-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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
Add go2rtc #34998
base: next
Are you sure you want to change the base?
Add go2rtc #34998
Conversation
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
📝 Walkthrough📝 WalkthroughWalkthroughThe changes introduce the Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant HomeAssistant
participant go2rtc
User->>HomeAssistant: Initiate go2rtc integration
HomeAssistant->>go2rtc: Connect to go2rtc instance
go2rtc-->>HomeAssistant: Provide WebRTC proxy for camera feeds
HomeAssistant-->>User: Stream camera feeds
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 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: 2
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
- source/_integrations/go2rtc.markdown (1 hunks)
🧰 Additional context used
🪛 LanguageTool
source/_integrations/go2rtc.markdown
[uncategorized] ~26-~26: “than” (as in: greater than) seems less likely than “then” (at that time, later on).
Context: ...nning {% term "Home Assistant Core" %}, than you need to setup a go2rtc instance and...(AI_HYDRA_LEO_CP_THAN_THEN)
[grammar] ~26-~26: The word “setup” is a noun. The verb is spelled with a space.
Context: ...me Assistant Core" %}, than you need to setup a go2rtc instance and provide the URL i...(NOUN_VERB_CONFUSION)
🪛 Markdownlint
source/_integrations/go2rtc.markdown
23-23: Expected: 0 or 2; Actual: 4
Trailing spaces(MD009, no-trailing-spaces)
27-27: null
Bare URL used(MD034, no-bare-urls)
🔇 Additional comments (2)
source/_integrations/go2rtc.markdown (2)
14-16
: LGTM: Clear introduction and proper use of templates.The introduction succinctly explains the purpose of the go2rtc integration, and the use of templates is correct. The inclusion of the configuration flow template is appropriate for guiding users through the setup process.
1-12
: Verify release version and consider specific codeowners.The YAML front matter looks good overall. However, there are two points to consider:
- The
ha_release
is set to 2024.11, which is in the future. Please confirm if this is the intended release version.- The
ha_codeowners
is set to '@home-assistant/core'. Consider adding specific individuals or a more specialized team as codeowners for this integration.To check if there are any other integrations with future release dates:
✅ Verification successful
ha_release
confirmed as intended for the upcoming release.The
ha_release
is set to2024.11
, which is appropriate for the planned release of the go2rtc integration. Additionally, no other integrations have future release dates, ensuring consistency across the codebase.Consider specifying more detailed codeowners to improve ownership clarity for this integration.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Check for other integrations with future release dates rg --type yaml 'ha_release: 202[4-9]' source/_integrations/Length of output: 61
If you are running {% term "Home Assistant Core" %}, than you need to setup a go2rtc instance and provide the URL in the config flow. | ||
Please refer to https://github.com/AlexxIT/go2rtc/ | ||
|
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.
Address grammatical issues and format URL properly.
There are a few issues in this section that need to be addressed:
- There's a grammatical error on line 26. "than" should be "then".
- On line 26, "setup" should be "set up" when used as a verb.
- The URL on line 27 should be formatted as a proper Markdown link.
Please apply the following changes:
- If you are running {% term "Home Assistant Core" %}, than you need to setup a go2rtc instance and provide the URL in the config flow.
- Please refer to https://github.com/AlexxIT/go2rtc/
+ If you are running {% term "Home Assistant Core" %}, then you need to set up a go2rtc instance and provide the URL in the config flow.
+ Please refer to [go2rtc GitHub repository](https://github.com/AlexxIT/go2rtc/) for more information.
These changes will improve the grammar and make the documentation more consistent with Markdown best practices.
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
If you are running {% term "Home Assistant Core" %}, than you need to setup a go2rtc instance and provide the URL in the config flow. | |
Please refer to https://github.com/AlexxIT/go2rtc/ | |
If you are running {% term "Home Assistant Core" %}, then you need to set up a go2rtc instance and provide the URL in the config flow. | |
Please refer to [go2rtc GitHub repository](https://github.com/AlexxIT/go2rtc/) for more information. |
🧰 Tools
🪛 LanguageTool
[uncategorized] ~26-~26: “than” (as in: greater than) seems less likely than “then” (at that time, later on).
Context: ...nning {% term "Home Assistant Core" %}, than you need to setup a go2rtc instance and...(AI_HYDRA_LEO_CP_THAN_THEN)
[grammar] ~26-~26: The word “setup” is a noun. The verb is spelled with a space.
Context: ...me Assistant Core" %}, than you need to setup a go2rtc instance and provide the URL i...(NOUN_VERB_CONFUSION)
🪛 Markdownlint
27-27: null
Bare URL used(MD034, no-bare-urls)
source/_integrations/go2rtc.markdown
Outdated
If you are running Home Assistant with | ||
|
||
- {% term "Home Assistant Operating System" %} | ||
- {% term "Home Assistant Supervised" %} | ||
- {% term "Home Assistant Container" %} | ||
|
||
than the go2rtc binary is already preinstalled in the container and the config flow will pick it automatically. |
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.
Fix grammatical error and remove trailing space.
The installation instructions are clear, but there are two minor issues to address:
- There's a grammatical error on line 24. "than" should be "then".
- There's a trailing space on line 23.
Please apply the following changes:
- than the go2rtc binary is already preinstalled in the container and the config flow will pick it automatically.
+ then the go2rtc binary is already preinstalled in the container and the config flow will pick it automatically.
Also, remove the trailing space at the end of line 23.
Committable suggestion was skipped due to low confidence.
🧰 Tools
🪛 Markdownlint
23-23: Expected: 0 or 2; Actual: 4
Trailing spaces(MD009, no-trailing-spaces)
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.
Thank you, @edenhaus 👍
|
||
than the go2rtc binary is already preinstalled in the container and the config flow will pick it automatically. | ||
|
||
If you are running {% term "Home Assistant Core" %}, than you need to setup a go2rtc instance and provide the URL in the config flow. |
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.
If you are running {% term "Home Assistant Core" %}, than you need to setup a go2rtc instance and provide the URL in the config flow. | |
If you are running {% term "Home Assistant Core" %}, you need to setup a go2rtc instance and provide the URL in the config flow. |
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.
We should explain a bit more what URL this is.
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
Co-authored-by: Martin Hjelmare <[email protected]>
Proposed change
Add go2rtc documentation
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.next
branch.Summary by CodeRabbit
New Features
go2rtc
integration, allowing users to connect to a WebRTC proxy for camera feeds.Documentation
go2rtc
integration.