Skip to content

switch to use debug_session.start for no-config debug telemetry #658

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Mar 7, 2025
Merged

switch to use debug_session.start for no-config debug telemetry #658

merged 2 commits into from
Mar 7, 2025

Conversation

eleanorjboyd
Copy link
Member

  • remove newly added telemetry event for no-config debugging
  • switch to using existing debug_session.start event with new trigger for no-config debug

@eleanorjboyd eleanorjboyd added the debt Covers everything internal: CI, testing, refactoring of the codebase, etc. label Mar 7, 2025
@eleanorjboyd eleanorjboyd self-assigned this Mar 7, 2025
@vs-code-engineering VS Code Engineering
Copy link

⚠️ This PR originates from a fork. Due to security restrictions, pipelines from forks are no longer triggered automatically. Learn more.

If the changes appear safe, you can manually trigger the pipeline by commenting /AzurePipelines run.

@vs-code-engineering vs-code-engineering bot added this to the March 2025 milestone Mar 7, 2025
@eleanorjboyd eleanorjboyd merged commit c3c7a41 into microsoft:main Mar 7, 2025
16 checks passed
@eleanorjboyd eleanorjboyd deleted the silly-seagull branch March 7, 2025 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Tyriar Tyriar Tyriar approved these changes

Assignees

@eleanorjboyd eleanorjboyd

Labels
debt Covers everything internal: CI, testing, refactoring of the codebase, etc.
Projects
None yet
Milestone
March 2025
Development

Successfully merging this pull request may close these issues.
2 participants
@eleanorjboyd @Tyriar