[contributing.md] use correct path separator in linux commands#16591
[contributing.md] use correct path separator in linux commands#16591mikeharder wants to merge 1 commit intomicrosoft:mainfrom
Conversation
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 16591Or
iex "& { $(irm https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 16591" |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the contributing guide to use the correct shell invocation syntax for macOS/Linux build scripts.
Changes:
- Fixes macOS/Linux build command from PowerShell-style
.\build.shto POSIX./build.sh - Applies the same fix to the
-packexample command
| ## Build the repo | ||
|
|
||
| `.\build.sh` (macOS and Linux) or `.\build.cmd` (Windows) | ||
| `./build.sh` (macOS and Linux) or `.\build.cmd` (Windows) |
There was a problem hiding this comment.
On Windows, the .\\build.cmd command is shown with a single backslash (e.g., .\build.cmd). In Markdown this is fine, but consider using consistent escaping/formatting to avoid copy/paste issues in some renderers (e.g., keep as-is but ensure other docs don’t use \\ or mixed forms). If the repo targets PowerShell, explicitly labeling these as 'PowerShell' vs 'bash/zsh' can reduce confusion.
|
|
||
| To do so simply execute: | ||
| `.\build.sh -pack` (macOS and Linux) or `.\build.cmd -pack` (Windows) | ||
| `./build.sh -pack` (macOS and Linux) or `.\build.cmd -pack` (Windows) |
There was a problem hiding this comment.
On Windows, the .\\build.cmd command is shown with a single backslash (e.g., .\build.cmd). In Markdown this is fine, but consider using consistent escaping/formatting to avoid copy/paste issues in some renderers (e.g., keep as-is but ensure other docs don’t use \\ or mixed forms). If the repo targets PowerShell, explicitly labeling these as 'PowerShell' vs 'bash/zsh' can reduce confusion.
Description
use correct path separator in linux commands
Checklist
<remarks />and<code />elements on your triple slash comments?aspire.devissue: