Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Avoid redefining AutoInsertTargetBranch - #12603

#12603
Merged
YuliiaKovalova merged 1 commit into
vs18.0dotnet/msbuild:vs18.0from
fix-insertion-yaml-for-18dotnet/msbuild:fix-insertion-yaml-for-18Copy head branch name to clipboard
Oct 6, 2025
Merged

Avoid redefining AutoInsertTargetBranch#12603
YuliiaKovalova merged 1 commit into
vs18.0dotnet/msbuild:vs18.0from
fix-insertion-yaml-for-18dotnet/msbuild:fix-insertion-yaml-for-18Copy head branch name to clipboard

Conversation

@rainersigwald

Copy link
Copy Markdown
Member

Fixes an error

/azure-pipelines/vs-insertion.yml (Line: 94, Col: 7): 'value' is already defined

That arose because we needed an elseif after the new clause.

Fixes an error

    /azure-pipelines/vs-insertion.yml (Line: 94, Col: 7): 'value' is already defined

That arose because we needed an elseif after the new clause.
Copilot AI review requested due to automatic review settings October 3, 2025 17:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 YAML syntax error in the Azure Pipelines configuration where a variable was being redefined due to an incorrect conditional statement structure.

  • Changes an if statement to elseif to properly chain conditional variable assignments
  • Prevents the "AutoInsertTargetBranch" variable from being redefined multiple times

@rainersigwald

Copy link
Copy Markdown
Member Author

I'll cancel the build so we don't need to bump the version since this file isn't used in actual builds.

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.

6 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.