Skip to content

Support for msbuild/build event#26427

Merged
marcpopMSFT merged 5 commits into
dotnet:mainfrom
rokonec:rokonec/msbuild-build-telemetry
Jul 20, 2022
Merged

Support for msbuild/build event#26427
marcpopMSFT merged 5 commits into
dotnet:mainfrom
rokonec:rokonec/msbuild-build-telemetry

Conversation

@rokonec
Copy link
Copy Markdown
Member

@rokonec rokonec commented Jul 4, 2022

Support for telemetry emitted by dotnet/msbuild#7778

This telemetry is supposed to be sent with every build, including base measurements and properties usable to understand usage of MSBuild.

This PR also fixes bug causing not sending telemetry from msbuild - introduced during changes of SDK calling msbuild.dll inproc.

NOTE: Although it is related to MSBuild server it could be merged into SDK before MSBuild server is released.

@ghost
Copy link
Copy Markdown

ghost commented Jul 4, 2022

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@rokonec rokonec requested a review from baronfel July 4, 2022 15:32
@rokonec rokonec self-assigned this Jul 4, 2022
Copy link
Copy Markdown
Member

@baronfel baronfel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems pretty straightforward - glad that it was easy to plumb through here.

Comment thread src/Cli/dotnet/commands/dotnet-msbuild/MSBuildLogger.cs Outdated
Comment thread src/Cli/dotnet/commands/dotnet-msbuild/MSBuildLogger.cs Outdated
@baronfel
Copy link
Copy Markdown
Member

Anything blocking us from merging this @rokonec?

@baronfel baronfel requested a review from dsplaisted July 20, 2022 13:42
@marcpopMSFT marcpopMSFT merged commit f96d21a into dotnet:main Jul 20, 2022
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.

3 participants