Skip to content

[release/v7.4] Bring Release Changes from v7.6.0-preview.6#26910

Merged
TravisEz13 merged 2 commits intoPowerShell:release/v7.4from
TravisEz13:backport/release/v7.4/26627-9ef5ec473
Feb 26, 2026
Merged

[release/v7.4] Bring Release Changes from v7.6.0-preview.6#26910
TravisEz13 merged 2 commits intoPowerShell:release/v7.4from
TravisEz13:backport/release/v7.4/26627-9ef5ec473

Conversation

@TravisEz13
Copy link
Member

Backport of #26627 to release/v7.4

Triggered by @TravisEz13 on behalf of @jshigetomi

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)

Updates build module (build.psm1) to skip dotnet-format installation in CI/CD (Azure DevOps) environments. Updates MSIX pipeline template with corrected Azure Pipeline variable references (using variables['NAME'] syntax instead of string interpolation). These changes prevent unnecessary tool installations in CI and ensure more reliable pipeline condition evaluation.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Verified cherry-pick completed successfully with conflict resolution. Changes are isolated to build scripts and CI/CD templates. The dotnet-format installation logic is now properly gated behind TF_BUILD environment variable check.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

Medium risk: Build system changes affecting CI/CD pipelines and dotnet tool installation, but changes are isolated and well-scoped. The dotnet-format installation is now properly skipped in CI environments (TF_BUILD), and MSIX pipeline conditions are fixed for better variable reference handling. No runtime impact on user-facing functionality.

Merge Conflicts

build.psm1 had one conflict due to missing dotnet-format installation code in release/v7.4. This was resolved by accepting the incoming change from the PR, which adds the Find-Dotnet call and wraps the dotnet-format installation in a TF_BUILD environment check to skip installation in CI environments.

…Shell#26627)

1. Fix the conditions used in `release-MSIX-Publish.yml`
2. Update `build.psm1` to not install dotnet format tool for ADO build.
@TravisEz13 TravisEz13 requested a review from a team as a code owner February 26, 2026 20:23
Copilot AI review requested due to automatic review settings February 26, 2026 20:23
@TravisEz13 TravisEz13 added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Feb 26, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Backports build/packaging pipeline changes from v7.6.0-preview.6 into release/v7.4, focusing on improving CI behavior and Azure Pipelines condition evaluation.

Changes:

  • Update Start-PSBootstrap to locate dotnet and gate .NET global tool installation (dotnet-format) based on TF_BUILD.
  • Fix MSIX publish task conditions to reference runtime variables via variables['NAME'] instead of $(NAME) macro expansion.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
build.psm1 Adds dotnet discovery and CI-aware gating around dotnet-format global tool installation in Start-PSBootstrap.
.pipelines/templates/release-MSIX-Publish.yml Updates task condition: expressions to use variables['STABLE'/'LTS'/'PREVIEW'] for reliable runtime evaluation.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@TravisEz13 TravisEz13 merged commit be13954 into PowerShell:release/v7.4 Feb 26, 2026
38 checks passed
adityapatwardhan pushed a commit to adityapatwardhan/PowerShell that referenced this pull request Feb 26, 2026
jshigetomi added a commit to jshigetomi/PowerShell that referenced this pull request Mar 2, 2026
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.

4 participants