-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update MSBuild.StructuredLogger #71576
Conversation
Job pushing the new version of MSBuild.StructuredLogger to dotnet-public: https://dev.azure.com/dnceng/internal/_build/results?buildId=2351338&view=results |
/azp run |
Commenter does not have sufficient privileges for PR 71576 in repo dotnet/roslyn |
Can anyone rerun the pipeline? The missed package is already in the feed
|
/azp run |
Azure Pipelines successfully started running 4 pipeline(s). |
@genlu is the |
@JanKrivanek The failing leg can be ignored at the moment; I think it might have only triggered due to the other /azp command that was given -- this is fine to merge assuming you're ready. |
@jasonmalinowski - thanks for details! Yes, it's ready :-) |
FYI @KirillOsenkov
Context
This is update to the latest version of MSBuild.StructuredLogger, that contains the forward compatibility reading support.
The initial MSBuild change to forward compatible logs reading is breaking (log data format needs to be adjusted in order to allow forward compatibile reading for the future) - hence updating the reading code ahead of MSBuild pushing the updated version of binlog writing (dotnet/msbuild#9307) is advisable.