From 8f1861557c0f5af16502cdfa07eea58173a8a7f8 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 32e4962f21e33..1af6f873912e2 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 'vs/base/common/amd'; import { CancellationError, isCancellationError } from 'vs/base/common/errors'; import { PromiseResult } from 'vs/base/common/observableInternal/promise'; -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 ea51e8361fc65..f769fe3039cca 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.