Skip to content

Update BootstrapSdkVersion to 10.0.107#13733

Merged
rainersigwald merged 1 commit into
mainfrom
raines/bump-boostrap-base-version-too
May 12, 2026
Merged

Update BootstrapSdkVersion to 10.0.107#13733
rainersigwald merged 1 commit into
mainfrom
raines/bump-boostrap-base-version-too

Conversation

@rainersigwald
Copy link
Copy Markdown
Member

Follow-up to #13721, which bumped the real SDK to 107 but didn't adjust this, leading to test flakiness #13732.

Work around that flakiness by updating the bootstrap base SDK to one that provides the same runtime the Arcade SDK requires.

Follow-up to #13721, which bumped the real SDK to 107 but didn't adjust this, leading to test flakiness #13732.

Work around that flakiness by updating the bootstrap base SDK to one that provides the same runtime the Arcade SDK requires.
Copilot AI review requested due to automatic review settings May 11, 2026 18:28
@rainersigwald rainersigwald requested a review from a team as a code owner May 11, 2026 18:28
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

This PR aligns MSBuild’s bootstrap SDK with the repository’s pinned .NET SDK (10.0.107) to avoid bootstrap/test flakiness caused by a runtime mismatch introduced by the Arcade dependency update in #13721.

Changes:

  • Bump BootstrapSdkVersion from 10.0.106 to 10.0.107 so the bootstrap layout includes the expected runtime/framework version.

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Review: Clean ✅

Reviewed against the 24-dimension MSBuild methodology. This is a minimal, well-scoped infrastructure fix.

  • Backwards Compatibility: No behavioral change — just aligning an internal version property. ✅
  • Correctness: BootstrapSdkVersion (10.0.106 → 10.0.107) now matches the dotnet SDK version in global.json (10.0.107), which was bumped in #13721. This eliminates the runtime mismatch that caused test flakiness (#13732). ✅
  • Scope Discipline: Single-concern PR, exactly one line changed. ✅
  • Build Infrastructure: No impact on public API, performance, or external consumers. ✅

No issues found.

Generated by Expert Code Review (on open) for issue #13733 · ● 1.1M

@rainersigwald
Copy link
Copy Markdown
Member Author

This failed with the flaky test we're trying to fix in #13704. That failed with this error. I am merging on red to unstick that.

@rainersigwald rainersigwald merged commit e6faff8 into main May 12, 2026
17 of 19 checks passed
@rainersigwald rainersigwald deleted the raines/bump-boostrap-base-version-too branch May 12, 2026 15:35
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