From 015e2c1a9e331270056f4259d4bab183609c0901 Mon Sep 17 00:00:00 2001 From: Kitson Kelly Date: Mon, 25 Jul 2022 07:11:33 +1000 Subject: [PATCH] fix(lsp): remove CompletionInfo.flags (#15288) Fixes: #15287 --- cli/lsp/tsc.rs | 16 +++------------- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a/cli/lsp/tsc.rs b/cli/lsp/tsc.rs index b701893df4..384a2d1819 100644 --- a/cli/lsp/tsc.rs +++ b/cli/lsp/tsc.rs @@ -1961,23 +1961,13 @@ impl CompletionEntryDetails { } } -#[derive(Debug, Deserialize_repr, Serialize_repr)] -#[repr(u32)] -pub enum CompletionInfoFlags { - None = 0, - MayIncludeAutoImports = 1, - IsImportStatementCompletion = 2, - IsContinuation = 4, - ResolvedModuleSpecifiers = 8, - ResolvedModuleSpecifiersBeyondLimit = 16, - MayIncludeMethodSnippets = 32, -} - #[derive(Debug, Deserialize, Serialize)] #[serde(rename_all = "camelCase")] pub struct CompletionInfo { entries: Vec, - flags: Option, + // this is only used by Microsoft's telemetrics, which Deno doesn't use and + // there are issues with the value not matching the type definitions. + // flags: Option, is_global_completion: bool, is_member_completion: bool, is_new_identifier_location: bool,