Skip to content

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

Merged
TravisEz13 merged 1 commit intoPowerShell:release/v7.4from
TravisEz13:backport/release/v7.4/26427-1fd3ea83
Feb 10, 2026
Merged

[release/v7.4] Fix condition syntax for StoreBroker package tasks in MSIX pipeline#26434
TravisEz13 merged 1 commit intoPowerShell:release/v7.4from
TravisEz13:backport/release/v7.4/26427-1fd3ea83

Conversation

@TravisEz13
Copy link
Member

@TravisEz13 TravisEz13 commented Nov 12, 2025

Backport of #26427 to release/v7.4

Triggered by @TravisEz13 on behalf of @TravisEz13

Original CL Label: CL-BuildPackaging

/cc @PowerShell/powershell-maintainers

Impact

Tooling Impact

  • Required tooling change

Fixes OneBranch condition syntax in MSIX pipeline that prevents StoreBroker package tasks from running correctly on release branches.

Regression

  • No

This is a tooling fix for pipeline configuration, not fixing a regression.

Testing

Verified by:

  1. Analyzing the OneBranch condition syntax requirements from the instruction file added in the PR
  2. Confirming the fix addresses the specific syntax issue in StoreBroker package tasks
  3. Cherry-pick applied cleanly to release/v7.4 branch

Risk

  • High
  • Medium
  • Low

High risk as it modifies build pipeline configuration, but necessary to ensure proper package creation on release branches. Not taking this change would prevent StoreBroker packages from being created correctly. The fix follows documented OneBranch condition syntax patterns and has been validated in the main branch.

@TravisEz13 TravisEz13 requested a review from a team as a code owner November 12, 2025 18:48
Copilot AI review requested due to automatic review settings November 12, 2025 18:48
@TravisEz13 TravisEz13 added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Nov 12, 2025
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 a fix for OneBranch condition syntax in the MSIX pipeline from the main branch to release/v7.4. The fix corrects the variable reference pattern in StoreBroker package task conditions to use the proper variables['VAR'] syntax instead of the incorrect '$(VAR)' string expansion syntax.

Key changes:

  • Updates two condition expressions in package-create-msix.yml to use correct OneBranch variable reference syntax
  • Adds comprehensive instruction documentation for OneBranch condition syntax patterns

Reviewed Changes

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

File Description
.pipelines/templates/package-create-msix.yml Corrects condition syntax from '$(PREVIEW)' to variables['PREVIEW'] and similar for STABLE/LTS conditions
.github/instructions/onebranch-condition-syntax.instructions.md Adds detailed instruction file documenting correct OneBranch condition syntax patterns, examples, and troubleshooting guidance

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@microsoft-github-policy-service microsoft-github-policy-service bot added the Review - Needed The PR is being reviewed label Nov 20, 2025
@microsoft-github-policy-service
Copy link
Contributor

This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days.
Maintainer, please provide feedback and/or mark it as Waiting on Author

@TravisEz13 TravisEz13 enabled auto-merge (squash) February 10, 2026 17:11
@TravisEz13 TravisEz13 merged commit c2057ed into PowerShell:release/v7.4 Feb 10, 2026
42 of 43 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 Review - Needed The PR is being reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants