Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Expose telemetry log to extension API.#7939

Merged
efritz merged 5 commits intomasterfrom
extension-telemetry
Jan 22, 2020
Merged

Expose telemetry log to extension API.#7939
efritz merged 5 commits intomasterfrom
extension-telemetry

Conversation

@efritz
Copy link
Contributor

@efritz efritz commented Jan 22, 2020

Add logTelemetryEvent command to the extension API so that extensions can send explicit anonymized events to the instance.

This will be used first by basic code intel to send events when a user receives precise or fuzzy code intel results. See sourcegraph/code-intel-extensions#193 for usage.

@codecov
Copy link

codecov bot commented Jan 22, 2020

Codecov Report

Merging #7939 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #7939      +/-   ##
==========================================
- Coverage   40.73%   40.72%   -0.01%     
==========================================
  Files        1279     1279              
  Lines       66590    66595       +5     
  Branches     6320     6321       +1     
==========================================
  Hits        27123    27123              
- Misses      36991    36996       +5     
  Partials     2476     2476
Impacted Files Coverage Δ
shared/src/commands/commands.ts 17.5% <0%> (-2.5%) ⬇️

@efritz efritz marked this pull request as ready for review January 22, 2020 16:23
@efritz efritz requested a review from a team January 22, 2020 16:23
@efritz efritz merged commit 09c20d0 into master Jan 22, 2020
@efritz efritz deleted the extension-telemetry branch January 22, 2020 19:08
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.

2 participants

Comments