Skip to content

Commit 4127be2

Browse files
Enable new telemetry by default
1 parent ce29768 commit 4127be2

File tree

2 files changed

+5
-6
lines changed

2 files changed

+5
-6
lines changed

extensions/ql-vscode/src/config.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -83,13 +83,8 @@ export const GLOBAL_ENABLE_TELEMETRY = new Setting(
8383
GLOBAL_TELEMETRY_SETTING,
8484
);
8585

86-
const ENABLE_NEW_TELEMETRY = new Setting(
87-
"enableNewTelemetry",
88-
TELEMETRY_SETTING,
89-
);
90-
9186
export function newTelemetryEnabled(): boolean {
92-
return ENABLE_NEW_TELEMETRY.getValue<boolean>();
87+
return true;
9388
}
9489

9590
// Distribution configuration

extensions/ql-vscode/test/vscode-tests/no-workspace/telemetry.test.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -394,6 +394,10 @@ describe("telemetry reporting", () => {
394394
});
395395

396396
describe("when new telementry is not enabled", () => {
397+
beforeEach(async () => {
398+
jest.spyOn(Config, "newTelemetryEnabled").mockReturnValue(false);
399+
});
400+
397401
it("should not send a ui-interaction telementry event", async () => {
398402
await telemetryListener.initialize();
399403

0 commit comments

Comments
 (0)