-
Notifications
You must be signed in to change notification settings - Fork 39
Update 7.1.0 release notes #266
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
WalkthroughThe release notes for Unraid OS 7.1.0 were updated to improve clarity and consistency in wording and link formatting. Adjustments include refining the description of Docker networking behavior with wireless enabled and softening the language regarding GPU passthrough requirements. No functional instructions or exported entities were changed. Changes
Possibly related PRs
Poem
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 (2)
docs/unraid-os/release-notes/7.1.0.md (2)
71-71
: Consistent formatting for network types
For improved readability and consistency with other technical terms, consider using inline code formatting foripvlan
andmacvlan
rather than bold. Example diff:-* …always use **ipvlan** (macvlan is not possible …) +* …always use `ipvlan` (`macvlan` is not possible …)
129-129
: Use active voice to clarify GPU passthrough warning
Rephrase in active voice to make the warning clearer to the user. For example:-* When the **Primary** GPU is assigned as passthrough for a VM, warn that it may not work without loading a compatible vBIOS. +* A warning now appears when you assign the **Primary** GPU as passthrough to a VM, indicating it may not work without a compatible vBIOS.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/unraid-os/release-notes/7.1.0.md
(2 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/unraid-os/release-notes/7.1.0.md
[duplication] ~68-~68: Possible typo: you repeated a word.
Context: ... * Only one wireless NIC is supported, wlan0 * wlan0 is not able to participate in a bond * ...
(ENGLISH_WORD_REPEAT_RULE)
[style] ~69-~69: As a shorter alternative for ‘able to’, consider using “can not”.
Context: ...ireless NIC is supported, wlan0 * wlan0 is not able to participate in a bond * Docker containe...
(BE_ABLE_TO)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: build
- GitHub Check: AWS Amplify Console Web Preview
🔇 Additional comments (1)
docs/unraid-os/release-notes/7.1.0.md (1)
63-63
: Verify external USB WiFi adapter link
Please confirm that the GitHub URL is correct and points to the intended file path—thehome/USB_WiFi_Adapters_that_are_supported_with_Linux_in-kernel_drivers.md
path looks unconventional. If this is an external reference, consider adding a concise link title for clarity.
This pull request is automatically being deployed by Amplify Hosting (learn more). |
Before Submitting This PR, Please Ensure You Have Completed The Following:
Summary by CodeRabbit