Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[release/v7.7.0-preview.1] PMC release: Use slash instead of back-slash for Linux container#27375

Merged
jshigetomi merged 1 commit into
PowerShell:release/v7.7.0-preview.1PowerShell/PowerShell:release/v7.7.0-preview.1from
jshigetomi:backport/release/v7.7.0-preview.1/27315-156906e05jshigetomi/PowerShell:backport/release/v7.7.0-preview.1/27315-156906e05Copy head branch name to clipboard
Apr 28, 2026
Merged

[release/v7.7.0-preview.1] PMC release: Use slash instead of back-slash for Linux container#27375
jshigetomi merged 1 commit into
PowerShell:release/v7.7.0-preview.1PowerShell/PowerShell:release/v7.7.0-preview.1from
jshigetomi:backport/release/v7.7.0-preview.1/27315-156906e05jshigetomi/PowerShell:backport/release/v7.7.0-preview.1/27315-156906e05Copy head branch name to clipboard

Conversation

@jshigetomi

Copy link
Copy Markdown
Collaborator

Backport of #27315 to release/v7.7.0-preview.1

Triggered by @jshigetomi on behalf of @daxian-dbw

Original CL Label: CL-BuildPackaging

/cc @PowerShell/powershell-maintainers

Impact

REQUIRED: Choose either Tooling Impact or Customer Impact (or both). At least one checkbox must be selected.

Tooling Impact

  • Required tooling change
  • Optional tooling change (include reasoning)

Fixes CredScan failure in PMC release pipeline on Linux containers by using forward slashes instead of backslashes in the suppress.json path.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Verified by original PR and successful merges to v7.4.15, v7.5.6, and v7.6.1 release branches. The fix corrects a path separator that caused CredScan to fail on Linux containers.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

Low risk — single character change (backslash to forward slash) in a pipeline YAML path variable; same fix already merged to v7.4, v7.5, and v7.6 release branches.

Merge Conflicts

Conflict in .pipelines/templates/release-prep-for-ev2.yml: The release branch had an additional ob_sdl_tsa_configFile variable not present on master. Resolution: applied the backslash-to-forward-slash fix for ob_sdl_credscan_suppressionsFile and preserved the extra ob_sdl_tsa_configFile variable from the release branch.

Copilot AI review requested due to automatic review settings April 28, 2026 19:26
@jshigetomi jshigetomi requested a review from a team as a code owner April 28, 2026 19:26
@jshigetomi jshigetomi added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Apr 28, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Backport to release/v7.7.0-preview.1 to fix CredScan in the PMC release pipeline when running in Linux containers by using Linux-compatible path separators for the CredScan suppressions file.

Changes:

  • Update ob_sdl_credscan_suppressionsFile to use forward slashes (/) instead of backslashes (\) in the suppressions file path.

@jshigetomi jshigetomi merged commit f1327fb into PowerShell:release/v7.7.0-preview.1 Apr 28, 2026
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.