PMC release: Use slash instead of back-slash for Linux container by daxian-dbw · Pull Request #27315 · PowerShell/PowerShell
Conversation
PR Summary
Use slash instead of back-slash for Linux container. Otherwise, the CreadScan step fails because it cannot find the file.
Copilot AI review requested due to automatic review settings
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Updates the EV2 release prep pipeline template to use POSIX-style paths so CredScan can locate the suppression file when running in a Linux container.
Changes:
- Switch
ob_sdl_credscan_suppressionsFilefrom Windows\separators to/separators for Linux execution.
| - group: 'packages.microsoft.com' | ||
| - name: ob_sdl_credscan_suppressionsFile | ||
| value: $(Build.SourcesDirectory)\PowerShell\.config\suppress.json | ||
| value: $(Build.SourcesDirectory)/PowerShell/.config/suppress.json |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider reusing the already-defined repoRoot variable here (e.g., $(repoRoot)/.config/suppress.json) instead of repeating $(Build.SourcesDirectory)/PowerShell. This reduces duplication and prevents future path changes from requiring multiple updates.
| value: $(Build.SourcesDirectory)/PowerShell/.config/suppress.json | |
| value: $(repoRoot)/.config/suppress.json |
Copilot uses AI. Check for mistakes.
Use slash instead of back-slash for Linux container
PMC release: Use slash instead of back-slash for Linux container
