Skip to content

Fix build break from concurrent changes w/api breaking change#52817

Merged
MichaelSimons merged 1 commit intodotnet:mainfrom
MichaelSimons:fix-build-break
Feb 3, 2026
Merged

Fix build break from concurrent changes w/api breaking change#52817
MichaelSimons merged 1 commit intodotnet:mainfrom
MichaelSimons:fix-build-break

Conversation

@MichaelSimons
Copy link
Member

This was caused by the breaking change in #52757 and #51962 being merged without updating to latest.

Copilot AI review requested due to automatic review settings February 3, 2026 20:55
@MichaelSimons MichaelSimons requested a review from a team February 3, 2026 20:56
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 resolves a build break caused by concurrent merges of two pull requests: PR #52757 (which refactored _testAssetsManager field to TestAssetsManager property) and PR #51962 (which added new tests using the old field name). The fix updates the two new test methods to use the renamed property.

Changes:

  • Updated references from _testAssetsManager to TestAssetsManager in two test methods added by PR #51962

@MichaelSimons
Copy link
Member Author

Force merging to address build break to unblock other work. The macOS (arm64) leg completed the build step which validates the build break was addressed.

@MichaelSimons MichaelSimons merged commit 1c90f4f into dotnet:main Feb 3, 2026
11 of 32 checks passed
@MichaelSimons MichaelSimons deleted the fix-build-break branch February 3, 2026 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants