Skip to content

[release/6.0] Update application-insights to 2.19.0#2996

Merged
MichaelSimons merged 5 commits intodotnet:release/6.0from
lbussell:update-application-insights
Aug 19, 2022
Merged

[release/6.0] Update application-insights to 2.19.0#2996
MichaelSimons merged 5 commits intodotnet:release/6.0from
lbussell:update-application-insights

Conversation

@lbussell
Copy link
Copy Markdown
Member

@lbussell lbussell requested a review from a team as a code owner August 18, 2022 00:39
@ghost ghost added the area-product-experience Improvements in the end-user's product experience label Aug 18, 2022
@MichaelSimons
Copy link
Copy Markdown
Member

@lbussell - Are these changes also needed? dotnet/source-build-externals#33

@lbussell
Copy link
Copy Markdown
Member Author

Are these changes also needed?

Trying to figure that out now - will add them if needed.

@MichaelSimons
Copy link
Copy Markdown
Member

IIRC this is only an issue in the tarball where it removes the online feeds from the known nuget.configs.

@MichaelSimons
Copy link
Copy Markdown
Member

...The result is the potential for prebuilts because NuGet non-determinism.

<UsageData>
<IgnorePatterns>
<UsagePattern IdentityGlob="*/*" />
<UsagePattern IdentityGlob="Microsoft.SourceBuild.Intermediate.*/*" />
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think we should be enabling prebuilt detection with these changes. There are likely other changes needed to get the repo level prebuilts clean. We don't want to block this PR on them.

@MichaelSimons MichaelSimons merged commit 7649ca7 into dotnet:release/6.0 Aug 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-product-experience Improvements in the end-user's product experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants