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
merge editor telemetry #164254
merge editor telemetry #164254
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may want to look into includes as this will help really reduce some of the duplicate properties which you share across events https://github.com/microsoft/vscode-telemetry-extractor/blob/main/documentation/typescript-code-annotations.md#Includes
|
||
manuallySolvedConflictCountThatEqualNone: number; | ||
manuallySolvedConflictCountThatEqualSmartCombine: number; | ||
manuallySolvedConflictCountThatEqualInput1: number; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if you could use a shared type for this so that the args and the data type being sent from telemetry don't need to be duplicated
owner: 'hediet'; | ||
comment: 'This event tracks when the user navigates to the previous conflict".'; | ||
}>('mergeEditor.action.goToPreviousConflict', { | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary empty line
owner: 'hediet'; | ||
comment: 'This event tracks when the user navigates to the next conflict".'; | ||
}>('mergeEditor.action.goToNextConflict', { | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary empty line
No description provided.