Results 173 comments of Jan Krivanek

Previous efforts: * https://github.com/dotnet/msbuild/issues/8969 * https://github.com/dotnet/msbuild/pull/7790 We should create a checklist for what needs to be updated - similar to https://github.com/dotnet/msbuild/blob/main/documentation/release-checklist.md

Those items might get resolved at the same time: * https://github.com/dotnet/msbuild/issues/9720 * https://github.com/dotnet/msbuild/issues/9366

Surayya`s PR (internal): https://devdiv.visualstudio.com/DevDiv/_git/DotNet-msbuild-Trusted/pullrequest/569790

Team is currently overloaded - so postponing this. Contributions are very welcome for this one.

Thank you @omajid for looking into our code and taking time to report this! I quickly peeked into this - all involved code paths have the pattern 'If not null...

Just suggestions: Btw. we should as well add de/serialization of `BuildCheckTracingEventArgs` and `BuildSubmissionStartedEventArgs` (the latter need to be introduced as well) - tracked under https://github.com/dotnet/msbuild/issues/10145 We might want to bundle...

> Does this result in big duplication for the console/text loggers at diag? That's the biggest concern off the top of my head right now. This should be identical. Console...

> I remember MuxLogger had these as false but somehow this was never the problem. I think something else is going on. > I cannot wrap my head around how...

Experimental VS insertion: https://devdiv.visualstudio.com/DevDiv/_git/VS/pullrequest/558711

Tested with VS with this PR inserted (installers: https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=9745016&view=ms.vss-build-web.run-extensions-tab) **tl;dr;:** The issue is present and resolved with this PR However it can have impact on legacy loggers that expect the...