Skip to content

Remove one of our out of proc analyzers#63180

Merged
CyrusNajmabadi merged 3 commits intodotnet:mainfrom
CyrusNajmabadi:removeTelemetry
Aug 5, 2022
Merged

Remove one of our out of proc analyzers#63180
CyrusNajmabadi merged 3 commits intodotnet:mainfrom
CyrusNajmabadi:removeTelemetry

Conversation

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor

Confirmation with owners of this telemetry indicate that this has not actually been helpful (esp. due to limiting data due to data size) and has not been checked in a very long time. Remove as it now serves no purpose and is actively impeding our goal of removing solution-crawler.

@CyrusNajmabadi CyrusNajmabadi requested a review from a team as a code owner August 3, 2022 19:48
@ghost ghost added the Area-IDE label Aug 3, 2022
@CyrusNajmabadi
Copy link
Copy Markdown
Contributor Author

Confirmed with Damien that this is not being used by them anymore.

Copy link
Copy Markdown
Member

@tmat tmat left a comment

Choose a reason for hiding this comment

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

:shipit:

@tmat
Copy link
Copy Markdown
Member

tmat commented Aug 4, 2022

Nice!

@CyrusNajmabadi CyrusNajmabadi merged commit e0b65a8 into dotnet:main Aug 5, 2022
@ghost ghost added this to the Next milestone Aug 5, 2022
@mavasani
Copy link
Copy Markdown
Contributor

mavasani commented Aug 5, 2022

Fantastic!

@CyrusNajmabadi CyrusNajmabadi deleted the removeTelemetry branch August 5, 2022 19:06
@dibarbet dibarbet modified the milestones: Next, 17.4 P2 Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants