Conversation
Adding new script.
WalkthroughAdds a “Bare Metal - Proxmox” section to HW installation docs, recommending a clean LXC/VM (Debian 13 or Ubuntu 24) and providing a wget-based install command for proxmox-install-netalertx.sh. Appended after Ubuntu 24 section. Contains minor typos; no code or behavior changes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/HW_INSTALL.md(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: docker_dev
| > [!NOTE] | ||
| > Use this on a clean LXC/VM for Debian 13 OR Ubuntu 24. | ||
| > The Scipt will detect OS and build acordingly. | ||
|
|
There was a problem hiding this comment.
Fix typos in the Proxmox note.
Please correct “Scipt” → “script” and “acordingly” → “accordingly” to keep the docs polished.
🤖 Prompt for AI Agents
In docs/HW_INSTALL.md around lines 79 to 82, fix the typos in the Proxmox note:
change “Scipt” to “script” and “acordingly” to “accordingly” so the sentence
reads correctly (e.g., “The script will detect OS and build accordingly.”).
|
FYI the base docker image is going through some changes. if you're depending on resources being somewhere, I'd recommend placing them in the install/proxmox folder. |
3 participants
Adding new script.
Summary by CodeRabbit