[main] Source code updates from dotnet/msbuild#4579
Merged
ViktorHofer merged 10 commits intomainfrom Feb 6, 2026
Merged
Conversation
Diff: https://github.com/dotnet/msbuild/compare/33d9fd7c7a003f0f45f2a51e5f4e751da255e416..8004741e56d5e783d03db6dd8d03c08126c0e73d From: dotnet/msbuild@33d9fd7 To: dotnet/msbuild@8004741 [[ commit created by automation ]]
Contributor
Author
|
Note PRs from original repository included in this codeflow update:
💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance. |
Diff: https://github.com/dotnet/msbuild/compare/8004741e56d5e783d03db6dd8d03c08126c0e73d..7744bc901ffc298e2ffa08f8235c873eb87e6c40 From: dotnet/msbuild@8004741 To: dotnet/msbuild@7744bc9 [[ commit created by automation ]]
Contributor
Author
|
Note PRs from original repository included in this codeflow update:
💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance. |
Diff: https://github.com/dotnet/msbuild/compare/7744bc901ffc298e2ffa08f8235c873eb87e6c40..93241074c73270a5824af855447b24365a425892 From: dotnet/msbuild@7744bc9 To: dotnet/msbuild@9324107 [[ commit created by automation ]]
Contributor
Author
|
Note PRs from original repository included in this codeflow update:
💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance. |
Member
|
@rainersigwald looks like dotnet/msbuild#13157 - more specifically the nullability change in SdkResolver.cs requires a reaction in sdk. PTAL |
rainersigwald
added a commit
to rainersigwald/msbuild
that referenced
this pull request
Feb 4, 2026
The doc comment explicitly mentions returning null and the SDK can do so, which caused a break in dotnet/dotnet#4579: src\sdk\src\Resolvers\Microsoft.NET.Sdk.WorkloadMSBuildSdkResolver\WorkloadSdkResolver.cs(35,36): error CS8764: Nullability of return type doesn't match overridden member (possibly because of nullability attributes).
Member
|
Indeed -- dotnet/msbuild#13197 should fix. Thanks for the ping! |
Member
|
Thanks. Btw you could have fixed this directly here in this PR and the change would have then flown back to msbuild. Just a note for the future in case you want to go that route. |
rainersigwald
added a commit
to dotnet/msbuild
that referenced
this pull request
Feb 4, 2026
The doc comment explicitly mentions returning null and the SDK can do so, which caused a break in dotnet/dotnet#4579: src\sdk\src\Resolvers\Microsoft.NET.Sdk.WorkloadMSBuildSdkResolver\WorkloadSdkResolver.cs(35,36): error CS8764: Nullability of return type doesn't match overridden member (possibly because of nullability attributes). Follow up to #13157 / 0172c58.
Diff: https://github.com/dotnet/msbuild/compare/93241074c73270a5824af855447b24365a425892..da989a5a30e59dfa0e22c0cc0e319210ed592705 From: dotnet/msbuild@9324107 To: dotnet/msbuild@da989a5 [[ commit created by automation ]]
Contributor
Author
|
Note PRs from original repository included in this codeflow update:
💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance. |
Diff: https://github.com/dotnet/msbuild/compare/da989a5a30e59dfa0e22c0cc0e319210ed592705..54d87c53bfa6554edc550216ff14874754bc2a5f From: dotnet/msbuild@da989a5 To: dotnet/msbuild@54d87c5 [[ commit created by automation ]]
Contributor
Author
|
Note PRs from original repository included in this codeflow update:
💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance. |
ViktorHofer
approved these changes
Feb 6, 2026
JanProvaznik
pushed a commit
to JanProvaznik/msbuild
that referenced
this pull request
Feb 25, 2026
The doc comment explicitly mentions returning null and the SDK can do so, which caused a break in dotnet/dotnet#4579: src\sdk\src\Resolvers\Microsoft.NET.Sdk.WorkloadMSBuildSdkResolver\WorkloadSdkResolver.cs(35,36): error CS8764: Nullability of return type doesn't match overridden member (possibly because of nullability attributes). Follow up to dotnet#13157 / 0172c58.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Note
This is a codeflow update. It may contain both source code changes from
the source repo
as well as dependency updates. Learn more here.
This pull request brings the following source code changes
From https://github.com/dotnet/msbuild
Diff the source with this PR branch