Skip to content

docs: move uninstall command into README uninstall section#435

Merged
miyoungc merged 1 commit intomainfrom
fix/readme-uninstall-section
Mar 19, 2026
Merged

docs: move uninstall command into README uninstall section#435
miyoungc merged 1 commit intomainfrom
fix/readme-uninstall-section

Conversation

@kjw3
Copy link
Copy Markdown
Contributor

@kjw3 kjw3 commented Mar 19, 2026

Summary

  • remove the misplaced uninstall curl snippet from the install section
  • update the dedicated README uninstall section to use the curl uninstall command
  • keep the install section focused on install-only steps

Testing

  • reviewed README structure on current main
  • verified the uninstall command now appears only under ### Uninstall

Summary by CodeRabbit

  • Documentation
    • Simplified uninstall documentation. Instructions now use a unified remote script execution method via curl, providing a more consistent and straightforward uninstall experience while removing local script references.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 19, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 6d67787e-b2ec-4f34-806c-42804086b53f

📥 Commits

Reviewing files that changed from the base of the PR and between bf0642c and 666ab2d.

📒 Files selected for processing (1)
  • README.md

📝 Walkthrough

Walkthrough

The README.md documentation was updated to simplify uninstall instructions. A standalone local script reference was removed, and the uninstall guidance now consistently directs users to execute the remote script via curl piped to bash, rather than running a local script file.

Changes

Cohort / File(s) Summary
Documentation Updates
README.md
Removed standalone uninstall script snippet and updated the Uninstall section to use direct remote script invocation (curl -fsSL .../uninstall.sh | bash) instead of running a local script (./uninstall.sh).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A hop, skip, and a script so neat,
Local files now obsolete,
With curl and bash, remote we go,
Uninstall's path now clean to flow.
🐰✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: moving the uninstall command from the install section into the dedicated Uninstall section of the README.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/readme-uninstall-section
📝 Coding Plan
  • Generate coding plan for human review comments

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

@kjw3 kjw3 requested a review from miyoungc March 19, 2026 19:57
@miyoungc miyoungc merged commit 6ca7d37 into main Mar 19, 2026
5 checks passed
Ryuketsukami pushed a commit to Ryuketsukami/NemoClaw that referenced this pull request Mar 24, 2026
jessesanford pushed a commit to jessesanford/NemoClaw that referenced this pull request Mar 24, 2026
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