Skip to content

[release/v7.5] Fix condition syntax for StoreBroker package tasks in MSIX pipeline#26819

Merged
daxian-dbw merged 1 commit intoPowerShell:release/v7.5from
daxian-dbw:backport/release/v7.5/26427-1fd3ea835
Feb 14, 2026
Merged

[release/v7.5] Fix condition syntax for StoreBroker package tasks in MSIX pipeline#26819
daxian-dbw merged 1 commit intoPowerShell:release/v7.5from
daxian-dbw:backport/release/v7.5/26427-1fd3ea835

Conversation

@daxian-dbw
Copy link
Member

Backport of #26427 to release/v7.5

Triggered by @daxian-dbw on behalf of @TravisEz13

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 condition syntax to use variables['VAR'] instead of string expansion

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Fix validated in 7.4 and 7.6 releases. Ensures StoreBroker tasks use correct condition evaluation.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

Simple fix for condition syntax in MSIX pipeline. Uses proper OneBranch variable reference syntax. Successfully backported to 7.4 and 7.6 branches.

@daxian-dbw daxian-dbw requested a review from a team as a code owner February 13, 2026 23:19
Copilot AI review requested due to automatic review settings February 13, 2026 23:19
@daxian-dbw daxian-dbw added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Feb 13, 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

This PR backports condition syntax fixes from #26427 to the release/v7.5 branch. It corrects the Azure Pipelines (OneBranch) condition syntax for StoreBroker package tasks in the MSIX pipeline to use the proper variables['VAR'] pattern instead of the incorrect '$(VAR)' string expansion pattern. The fix has been validated in 7.4 and 7.6 releases.

Changes:

  • Fixed two condition expressions in the MSIX pipeline to use correct OneBranch variable reference syntax
  • Added comprehensive documentation explaining OneBranch condition syntax patterns and best practices

Reviewed changes

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

File Description
.pipelines/templates/package-create-msix.yml Corrected condition syntax for StoreBroker package tasks (Preview and Stable/LTS) to use variables['VAR'] instead of '$(VAR)'
.github/instructions/onebranch-condition-syntax.instructions.md New documentation file providing comprehensive guidance on correct OneBranch condition syntax patterns, common scenarios, troubleshooting, and best practices

@daxian-dbw daxian-dbw merged commit d0a41f8 into PowerShell:release/v7.5 Feb 14, 2026
21 checks passed
@daxian-dbw daxian-dbw deleted the backport/release/v7.5/26427-1fd3ea835 branch February 14, 2026 00:24
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