Skip to content

Version bump check fix#13461

Merged
MichalPavlik merged 2 commits intomainfrom
dev/mipavlik/fix-version-bump-check-2
Mar 30, 2026
Merged

Version bump check fix#13461
MichalPavlik merged 2 commits intomainfrom
dev/mipavlik/fix-version-bump-check-2

Conversation

@MichalPavlik
Copy link
Copy Markdown
Member

Fixes #13451

Context

Version bump check didn't work. This change should fix that.

Changes Made

Node selection logic was changed.

Testing

Change tested in release branch: https://dev.azure.com/dnceng-public/public/_build/results?buildId=1355155&view=logs&j=6fe8bf02-8a7a-56a7-e3fd-913e2045a877&t=e87b56af-cb98-594d-acd9-14509c9c5689

Notes

Copy link
Copy Markdown
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

Fixes the release-branch “version bump check” by correcting how the pipeline reads DotNetFinalVersionKind from eng/Versions.props, ensuring the servicing-branch bump validation runs when the repo is in a release state (including VS Insiders scenarios described in #13451).

Changes:

  • Update the version-kind read logic in the CI pipeline to reliably extract the DotNetFinalVersionKind element value.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@MichalPavlik MichalPavlik merged commit 66645d8 into main Mar 30, 2026
10 checks passed
@MichalPavlik MichalPavlik deleted the dev/mipavlik/fix-version-bump-check-2 branch March 30, 2026 14:31
dfederm pushed a commit to dfederm/msbuild that referenced this pull request Apr 9, 2026
Fixes dotnet#13451

### Context
Version bump check didn't work. This change should fix that.

### Changes Made
Node selection logic was changed.

### Testing
Change tested in release branch:
https://dev.azure.com/dnceng-public/public/_build/results?buildId=1355155&view=logs&j=6fe8bf02-8a7a-56a7-e3fd-913e2045a877&t=e87b56af-cb98-594d-acd9-14509c9c5689

### Notes

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

Bump-version check on release branches is wrong for VS Insiders

3 participants