Closed ladipro closed 1 month ago
Published https://github.com/KirillOsenkov/MSBuildStructuredLog/releases/tag/v2.2.243 - please try it out to make sure it all works as expected. Thanks!
@KirillOsenkov I have tested it on version 2.2.248 and all seems to be good. Please confirm that it's OK to merge the MSBuild PR now.
yes, go ahead
https://github.com/dotnet/msbuild/pull/10130 is making MSBuild log
TaskParameterEventArgs
for all task parameters, including all output parameters. This PR adds support for this format change to the viewer.Changes
TaskParameterEventArgs
used for all parameters.Testing
Old binlog with new viewer - no changes.
New binlog with old viewer - shows the
Skipped some data unknown to this version ...
warning and misplaces output parameters assigned to properties. They are incorrectly displayed underOutputItems
withN/A
name.This is assumed to be an acceptable break for users who haven't updated to the new version.
This is just a quick proof-of-concept as I'm not sure what the best way of presenting this data is or if there's appetite for enriching the UI at all. We could certainly not show the parameter name anywhere for now.