Skip to content
This repository has been archived by the owner on Oct 4, 2021. It is now read-only.

[IDE] Track startup metadata in one place #7355

Closed
wants to merge 1 commit into from
Closed

[IDE] Track startup metadata in one place #7355

wants to merge 1 commit into from

Conversation

iainx
Copy link
Contributor

@iainx iainx commented Mar 19, 2019

Fixes VSTS #823325 The startup telemetry metadata code should be consolidated into one class

@iainx iainx requested a review from slluis as a code owner March 19, 2019 17:15
@iainx
Copy link
Contributor Author

iainx commented Mar 19, 2019

@monojenkins backport release-8.0

@iainx
Copy link
Contributor Author

iainx commented Mar 20, 2019

@slluis @mrward fixed

@iainx iainx requested review from slluis and mrward March 27, 2019 10:39
@Therzok
Copy link
Contributor

Therzok commented Mar 27, 2019

@iainx maybe this should be rebased on top of master-vnext, as some stuff has changed there and it conflicts.

Fixes VSTS #823325 The startup telemetry metadata code should be consolidated into one class
@iainx
Copy link
Contributor Author

iainx commented Mar 27, 2019

@Therzok rebased against master for the moment.

@Therzok
Copy link
Contributor

Therzok commented Mar 28, 2019

Code changes including new workspace events LGTM!

@sevoku
Copy link
Member

sevoku commented May 23, 2019

This is already merged: 0331b37

@sevoku sevoku closed this May 23, 2019
@sevoku sevoku deleted the fix-823325 branch May 23, 2019 14:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants