docs: clarify Windows commands run in Command Prompt, not PowerShell

[SAY-5]

Description

Adds a short note in the Windows tab of both the "Verifying the Isaac Sim installation" sections (pre-built binaries and source installation guides) calling out that the Windows commands shown throughout the guide use Command Prompt (cmd.exe) syntax, not PowerShell, so commands like set and mklink work as documented.

Per @RandomOakForest in #5552:

Following up, if you have time for a PR, please submit it. The team will review it and will aim to merge it.

Fixes #5552

Type of change

• Documentation update

Screenshots

N/A (text-only addition).

Checklist

• I have read and understood the contribution guidelines
• I have run the pre-commit checks with ./isaaclab.sh --format
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• I have updated the changelog and the corresponding version in the extension's config/extension.toml file
• I have added my name to the CONTRIBUTORS.md or my name already exists there

Note: this is a docs-only change (two RST files, no code, no extension touched), so the pre-commit, tests, changelog and extension version items above are not applicable. Happy to also add a CONTRIBUTORS.md entry in a follow-up if maintainers prefer.

[greptile-apps]

Greptile Summary

This PR adds a short .. note:: directive in the Windows section of the "Verifying the Isaac Sim installation" heading in both the pre-built binaries and source installation guides, alerting users to run Windows commands in cmd.exe rather than PowerShell.

• Both RST files receive the identical note block; wording is clear, technically accurate, and uses correct RST formatting (.. note:: with proper indentation).
• Placement is appropriate in both files — the cmd.exe-specific commands (set, mklink) appear immediately after the inserted note in each guide.

Confidence Score: 5/5

Safe to merge — two RST files, no code touched, note content is accurate and well-formed.

Both changes are identical, minimal RST additions that correctly use the .. note:: directive with proper indentation. The note content (set and mklink not working in PowerShell) is factually correct, and placement immediately before the first batch of cmd.exe commands is appropriate in both guides.

No files require special attention.

Important Files Changed

Filename	Overview
docs/source/setup/installation/binaries_installation.rst	Adds a .. note:: block under "Verifying the Isaac Sim installation" clarifying that Windows tabs use cmd.exe syntax, not PowerShell. Placement is appropriate — the first set commands appear immediately after the note.
docs/source/setup/installation/source_installation.rst	Same note block added under "Verifying the Isaac Sim installation". The build.bat Windows command shown earlier in the "Building from source" section works fine in either shell, so the note's placement after that section is still accurate.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User opens Windows installation guide] --> B{Which shell?}
    B -->|cmd.exe| C[Commands work as documented\nset, mklink, etc.]
    B -->|PowerShell| D[Commands may fail\nor behave unexpectedly]
    E[New note block added here] --> B
    style E fill:#ffffcc,stroke:#cccc00
    style C fill:#ccffcc,stroke:#00cc00
    style D fill:#ffcccc,stroke:#cc0000

Loading

_{Reviews (1): Last reviewed commit: "docs: clarify Windows commands run in Co..." | Re-trigger Greptile}

[isaaclab-review-bot]

🤖 Isaac Lab Review Bot — Multi-Perspective Analysis

PR: #5736 — docs: clarify Windows commands run in Command Prompt, not PowerShell
Author: @SAY-5 | Commit: ec340bf9 | Files changed: 2

---

📋 Summary

This PR adds a .. note:: directive to both the binary and source installation guides, warning Windows users that the documented commands use Command Prompt (cmd.exe) syntax rather than PowerShell. This directly addresses the confusion reported in #5552.

---

✅ Verdict: LGTM

Aspect	Assessment
Accuracy	✅ Correct — set and mklink are indeed cmd.exe built-ins that behave differently (or fail) in PowerShell
Placement	✅ Well-placed — appears at the top of "Verifying the Isaac Sim installation" in both guides, before users encounter those commands
RST Syntax	✅ Valid .. note:: directive with correct indentation and blank lines
Consistency	✅ Identical wording in both files ensures a uniform experience
Completeness	✅ Covers both installation paths (binaries & source)
Grammar/Clarity	✅ Clean, concise, no issues

---

💡 Optional Suggestions (non-blocking)

1. Scope of note: The note says "throughout this guide" — since it's placed mid-page under "Verifying the Isaac Sim installation," users who scroll past it to later sections may miss it. Consider whether placing it even earlier (e.g., under the top-level prerequisites section) might catch more readers. However, the current placement is reasonable since set commands appear immediately after.

2. PowerShell equivalent hint: A brief parenthetical like "($env:VAR in PowerShell)" could help PowerShell users translate, but this is purely a nice-to-have and outside the scope of this PR.

---

📝 Update History

Commit	Change
0c91185e	Initial review — LGTM
88bae1b4	Merge from main (daqp bump, nightly-changelog CI, Pink IK fix) — no PR file changes
ec340bf9	Merge from main (RSL-RL checkpoint migration from #5903) — no PR file changes

---

Latest (ec340bf): This commit is another merge of main into the PR branch, bringing in the rsl-rl checkpoint deprecation handler from #5903. The PR's documentation files (binaries_installation.rst and source_installation.rst) are unchanged. Original LGTM assessment remains valid. ✅

[isaaclab-review-bot]

---

🔄 Incremental Update — 88bae1b4

New commits since last review: 3 (all merge activity from main)

Commit	Type	Description
f73c331	main merge	daqp pin bump to 0.8.5 (Pink IK fix)
504acae	main merge	CI nightly-changelog workflow fixes
88bae1b	branch merge	Merge branch 'main' into docs/windows-cmd-prompt

Files changed in PR content: 0

This is another routine merge bringing the feature branch up-to-date with main. The PR's documentation changes (the .. note:: directives in docs/source/setup/installation/binaries_installation.rst and docs/source/setup/installation/pip_installation.rst) remain unchanged.

Previous assessment stands: ✅ LGTM — no new issues introduced.