docs(manus): clarify installation instructions and prerequisites#463
docs(manus): clarify installation instructions and prerequisites#463Aniket-Manus wants to merge 3 commits intoNVIDIA:mainfrom
Conversation
|
📝 Docs preview is not auto-deployed for fork PRs. A maintainer with write access to |
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThe documentation for the Manus plugin was updated to clarify installation requirements. A note was added stating that the Manus plugin cannot be installed via standard pip packaging and must be built from source first, with a reference to the build documentation. The installation section structure was reorganized by introducing an "Automated (recommended)" subsection heading and updating the manual installation lead-in text to specify that manual steps follow source builds. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/source/device/manus.rst`:
- Line 33: The Sphinx cross-reference :doc:`build_from_source/index` in the
manus.rst text is pointing to a non-existent docs/source/build_from_source/ doc;
either create that documentation folder and an index.rst (or appropriate .rst)
under build_from_source so the :doc: target resolves, or update the reference in
manus.rst to the correct existing doc path (for example replace
:doc:`build_from_source/index` with the actual target name in the docs tree).
Ensure the referenced document has an index file and is included in the toctree
so Sphinx can build the link.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 631b925f-86df-42ee-aa5f-e8c5b662a402
📒 Files selected for processing (1)
docs/source/device/manus.rst
There was a problem hiding this comment.
Pull request overview
Updates the Manus device documentation to clarify that the Manus plugin must be built from source (not installed via standard pip packaging) and to reorganize the installation guidance for clarity.
Changes:
- Added an explicit note that Isaac Teleop must be built from source before using the Manus plugin.
- Adjusted the structure/wording of the automated and manual installation instructions.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Manus plugin requires building the Teleop repo from source. This has now been clarified in the docs
Summary by CodeRabbit