Skip to content

Update MicrosoftBuildVersion to 17.12.50#6870

Closed
Nigusu-Allehu wants to merge 3 commits intorelease/7.0.xfrom
backport/pr-6866-to-release/7.0.x
Closed

Update MicrosoftBuildVersion to 17.12.50#6870
Nigusu-Allehu wants to merge 3 commits intorelease/7.0.xfrom
backport/pr-6866-to-release/7.0.x

Conversation

@Nigusu-Allehu
Copy link
Copy Markdown
Member

Bug

Fixes:

Description

PR Checklist

  • Meaningful title, helpful description and a linked NuGet/Home issue
  • Added tests
  • Link to an issue or pull request to update docs if this PR changes settings, environment variables, new feature, etc.

@Nigusu-Allehu Nigusu-Allehu requested a review from a team as a code owner October 17, 2025 21:00
@zivkan
Copy link
Copy Markdown
Member

zivkan commented Oct 17, 2025

Please use a more meaningful PR title. I don't know what PR 6866 was and you didn't provide a link in the PR description. Also consider once this PR is merged and someone's looking at the branch history a commit message that helps understand what the change is will be much more useful than a message saying it's a copy of some other PR.

Copy link
Copy Markdown
Contributor

@donnie-msft donnie-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use the same title as the backported commit, and consider prefixing it with [release/7.0.x] or something similar to indicate it's a backport.

@Nigusu-Allehu Nigusu-Allehu changed the title Backport/pr 6866 to release/7.0.x Update MicrosoftBuildVersion to 17.12.50 Oct 18, 2025
@Nigusu-Allehu
Copy link
Copy Markdown
Member Author

This was a branch created by the backport worlflow and it looks like the commit hisotry contains commits that were squashed in the orginal commit. I will close this and cherry pick the fix and create a new PR

@zivkan zivkan deleted the backport/pr-6866-to-release/7.0.x branch January 7, 2026 00:00
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.

3 participants