Skip to content
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

Another attempt at getting TelemetryLogger working in VSCode #71734

Merged
merged 3 commits into from
Jan 20, 2024

Conversation

ToddGrun
Copy link
Contributor

This source links a couple files into devkit that use VSTelemetry to enable our TelemetryLogger API. Additionally, it adds a TelemetryLogger.Create call into VSCodeTelemetryLogger to actually create the TelemetryLogProvider

@ToddGrun ToddGrun requested a review from a team as a code owner January 19, 2024 22:49
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead labels Jan 19, 2024
@ToddGrun ToddGrun requested a review from dibarbet January 19, 2024 22:49
@ToddGrun ToddGrun force-pushed the dev/toddgrun/VSCodeTelemetryLogger branch from 2081046 to 1707ce3 Compare January 20, 2024 00:46
…enable our TelemetryLogger API. Additionally, it adds a TelemetryLogger.Create call into VSCodeTelemetryLogger to actually create the TelemetryLogProvider
@ToddGrun ToddGrun force-pushed the dev/toddgrun/VSCodeTelemetryLogger branch from 1707ce3 to 5293a74 Compare January 20, 2024 00:48
@ToddGrun ToddGrun force-pushed the dev/toddgrun/VSCodeTelemetryLogger branch from c53eed4 to f5ef0a2 Compare January 20, 2024 00:54
@ToddGrun ToddGrun merged commit 2fe96bc into dotnet:main Jan 20, 2024
28 checks passed
@ghost ghost added this to the Next milestone Jan 20, 2024
@Cosifne Cosifne modified the milestones: Next, 17.10 P1 Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants