Skip to content

Conversation

@TravisEz13
Copy link
Member

This PR fixes the condition syntax for StoreBroker package tasks in the MSIX pipeline to use proper OneBranch syntax.

The changes ensure that conditional expressions follow the correct OneBranch format for evaluating pipeline variables.

@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 11, 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 fixes condition syntax for two StoreBroker package tasks in the MSIX pipeline to use the correct OneBranch variable reference pattern, and adds a comprehensive instruction file documenting the proper OneBranch condition syntax for the repository.

Key Changes

  • Updated StoreBroker package conditions from '$(VARIABLE)' string expansion to variables['VARIABLE'] syntax
  • Added comprehensive OneBranch condition syntax instruction file with examples and best practices

Reviewed Changes

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

File Description
.pipelines/templates/package-create-msix.yml Fixed condition syntax for Preview and Stable/LTS StoreBroker package tasks to use proper variables['NAME'] pattern
.github/instructions/onebranch-condition-syntax.instructions.md Added comprehensive guide documenting correct OneBranch condition syntax with examples, troubleshooting tips, and best practices

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

@TravisEz13 TravisEz13 merged commit 1fd3ea8 into PowerShell:master Nov 12, 2025
36 of 37 checks passed
@TravisEz13 TravisEz13 deleted the fix-storebroker-condition-syntax branch November 12, 2025 17:35
TravisEz13 added a commit to TravisEz13/PowerShell that referenced this pull request Nov 12, 2025
adityapatwardhan pushed a commit to adityapatwardhan/PowerShell that referenced this pull request Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport-7.4.x-Migrated BackPort-7.5.x-Consider Backport-7.6.x-Migrated 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