Skip to content

Conversation

@gonzague
Copy link
Contributor

@gonzague gonzague commented Oct 3, 2025

Fixing references to the Ubuntu install script , the install script in the doc references a install.ubuntu24.sh script when the script is actually located at install.sh

Summary by CodeRabbit

  • Documentation
    • Updated installation guide to use a unified install.sh across both curl and wget methods.
    • Corrected download URLs, chmod command, and execution examples to match the new script name.
    • Removed outdated references to OS-specific script names for clarity.
    • Simplifies and standardizes the installation process for users following the guide.

Fixing references to the Ubuntu install script
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 3, 2025

Walkthrough

Replaces references to install.ubuntu24.sh with install.sh in HW installation documentation, updating curl and wget URLs and corresponding chmod and execution commands.

Changes

Cohort / File(s) Summary
Docs: install script rename
docs/HW_INSTALL.md
Updated installation commands to use install.sh instead of install.ubuntu24.sh for curl/wget, including chmod and execution steps.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I nibbled the docs, just a tiny bite,
Swapped a script name in the moonlit night.
Curl hops left, wget hops right—
install.sh now shines so bright.
Thump-thump approval, whiskers light! 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Title Check ✅ Passed The title clearly and concisely reflects the main change of updating script references in the HW_INSTALL.md documentation, specifying exactly what is being fixed and where. It accurately summarizes the core modification without extraneous detail, making it immediately understandable to reviewers. This phrasing helps teammates quickly grasp the purpose of the pull request.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@gonzague gonzague changed the title Update HW_INSTALL.md Fix install script references in HW_INSTALL.md Oct 3, 2025
Copy link
Contributor Author

@gonzague gonzague left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated title

@jokob-sk jokob-sk merged commit fc72abc into netalertx:main Oct 4, 2025
2 checks passed
@jokob-sk
Copy link
Collaborator

jokob-sk commented Oct 4, 2025

Thanks @gonzague 🙏

@coderabbitai coderabbitai bot mentioned this pull request Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants