From 03eb933857611ae2b0d98b6209d7d0d5667b8aec Mon Sep 17 00:00:00 2001 From: Huiwen Date: Fri, 6 Sep 2024 12:35:13 +0000 Subject: [PATCH] Remove treeSitterTelemetry --- .../services/treeSitter/treeSitterParserService.ts | 5 +++-- .../common/config/editorConfigurationSchema.ts | 12 ++++++------ 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/src/vs/editor/browser/services/treeSitter/treeSitterParserService.ts b/src/vs/editor/browser/services/treeSitter/treeSitterParserService.ts index 26a61c41c1afe..d65262972efbf 100644 --- a/src/vs/editor/browser/services/treeSitter/treeSitterParserService.ts +++ b/src/vs/editor/browser/services/treeSitter/treeSitterParserService.ts @@ -23,7 +23,7 @@ import { canASAR } from '../../../../base/common/amd.js'; import { CancellationError, isCancellationError } from '../../../../base/common/errors.js'; import { PromiseResult } from '../../../../base/common/observable.js'; -const EDITOR_TREESITTER_TELEMETRY = 'editor.experimental.treeSitterTelemetry'; +// const EDITOR_TREESITTER_TELEMETRY = 'editor.experimental.treeSitterTelemetry'; const MODULE_LOCATION_SUBPATH = `@vscode/tree-sitter-wasm/wasm`; const FILENAME_TREESITTER_WASM = `tree-sitter.wasm`; @@ -396,7 +396,8 @@ export class TreeSitterTextModelService extends Disposable implements ITreeSitte if (setting && setting.length > 0) { return setting; } else { - const expSetting = this._configurationService.getValue(EDITOR_TREESITTER_TELEMETRY); + // const expSetting = this._configurationService.getValue(EDITOR_TREESITTER_TELEMETRY); + const expSetting = false; if (expSetting) { return ['typescript']; } diff --git a/src/vs/editor/common/config/editorConfigurationSchema.ts b/src/vs/editor/common/config/editorConfigurationSchema.ts index e89269b9f218b..1a2e4a44eedbf 100644 --- a/src/vs/editor/common/config/editorConfigurationSchema.ts +++ b/src/vs/editor/common/config/editorConfigurationSchema.ts @@ -109,12 +109,12 @@ const editorConfiguration: IConfigurationNode = { description: nls.localize('editor.experimental.asyncTokenizationVerification', "Controls whether async tokenization should be verified against legacy background tokenization. Might slow down tokenization. For debugging only."), tags: ['experimental'], }, - 'editor.experimental.treeSitterTelemetry': { - type: 'boolean', - default: false, - markdownDescription: nls.localize('editor.experimental.treeSitterTelemetry', "Controls whether tree sitter parsing should be turned on and telemetry collected. Setting `editor.experimental.preferTreeSitter` for specific languages will take precedence."), - tags: ['experimental'] - }, + // 'editor.experimental.treeSitterTelemetry': { + // type: 'boolean', + // default: false, + // markdownDescription: nls.localize('editor.experimental.treeSitterTelemetry', "Controls whether tree sitter parsing should be turned on and telemetry collected. Setting `editor.experimental.preferTreeSitter` for specific languages will take precedence."), + // tags: ['experimental'] + // }, 'editor.language.brackets': { type: ['array', 'null'], default: null, // We want to distinguish the empty array from not configured.