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

More telemetry events #22171

Merged
merged 3 commits into from
Dec 18, 2024
Merged

More telemetry events #22171

merged 3 commits into from
Dec 18, 2024

Conversation

ConradIrwin
Copy link
Member

  • Convert more events to telemetry::event
  • And call events

Closes #ISSUE

Release Notes:

  • N/A

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Dec 17, 2024
@ConradIrwin ConradIrwin merged commit 94bfb93 into main Dec 18, 2024
13 checks passed
@ConradIrwin ConradIrwin deleted the more-telemetry-event branch December 18, 2024 05:17
helgemahrt pushed a commit to helgemahrt/zed that referenced this pull request Dec 18, 2024
- **Convert more events to telemetry::event**
- **And call events**

Closes #ISSUE

Release Notes:

- N/A
mrnugget added a commit that referenced this pull request Jan 10, 2025
Seems like #22171 accidentally removed this line.

Now it's back and completions are marked as rated again.
github-merge-queue bot pushed a commit that referenced this pull request Jan 10, 2025
Seems like #22171 accidentally removed this line.

Now it's back and completions are marked as rated again.

![screenshot-2025-01-10-10 56
32@2x](https://github.com/user-attachments/assets/c68bff1b-5b97-493e-9062-390876fd757c)

Release Notes:

- N/A
as-cii pushed a commit that referenced this pull request Jan 15, 2025
Seems like #22171 accidentally removed this line.

Now it's back and completions are marked as rated again.

![screenshot-2025-01-10-10 56
32@2x](https://github.com/user-attachments/assets/c68bff1b-5b97-493e-9062-390876fd757c)

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant