Skip to content

Conversation

@jjonescz
Copy link
Member

@jjonescz jjonescz commented Jan 7, 2026

The branch merge workflow seems to be completely broken in the sdk repo these days with errors like:

Run D:\a\sdk\sdk\arcade-repository\.github\workflows\scripts\inter-branch-merge.ps1 -RepoName sdk -RepoOwner dotnet -MergeFromBranch $env:GITHUB_REF_NAME -MergeToBranch main -ResetToTargetPaths "" -QuietComments -ResetToTargetPaths global.json;NuGet.config;eng/Version.Details.xml;eng/Version.Details.props;eng/common/*
inter-branch-merge.ps1: D:\a\_temp\99bd6d72-4688-4ba6-a447-0fd0ca3d76d0.ps1:2
Line |
   2 |  … ain -ResetToTargetPaths "" -QuietComments -ResetToTargetPaths global. …
     |                                              ~~~~~~~~~~~~~~~~~~~
     | Cannot bind parameter because parameter 'ResetToTargetPaths' is specified more than once. To provide multiple
     | values to parameters that can accept multiple values, use the array syntax. For example, "-parameter
     | value1,value2,value3".
Error: Process completed with exit code 1.

I believe this PR should fix that.

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 a critical issue in the inter-branch merge workflow that was causing automation failures due to duplicate PowerShell parameters. The workflow script was receiving -ResetToTargetPaths specified twice - once as an empty default value and again with actual file paths from the ExtraSwitches configuration.

Key Changes:

  • Extracted ResetToTargetPaths parameter from ExtraSwitches into a dedicated property across all merge flow configurations
  • Standardized configuration structure to prevent PowerShell parameter binding errors

@jjonescz
Copy link
Member Author

jjonescz commented Jan 7, 2026

/ba-g CI skipped

@jjonescz
Copy link
Member Author

jjonescz commented Jan 8, 2026

/ba-g CI skipped

@jjonescz jjonescz merged commit 38187ce into dotnet:main Jan 8, 2026
6 of 7 checks passed
@jjonescz jjonescz deleted the merge-flow-config branch January 8, 2026 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants