fix(lsp): don't commit registry completions on "/" (denoland#20902)

howqw · Oct 13, 2023 · bd238be · bd238be
1 parent 7599990
commit bd238be
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/cli/lsp/registries.rs b/cli/lsp/registries.rs
@@ -64,7 +64,7 @@ const COMPONENT: &percent_encoding::AsciiSet = &percent_encoding::CONTROLS
  .add(b'+')
  .add(b',');
 
-const REGISTRY_IMPORT_COMMIT_CHARS: &[&str] = &["\"", "'", "/"];
+const REGISTRY_IMPORT_COMMIT_CHARS: &[&str] = &["\"", "'"];
 
 static REPLACEMENT_VARIABLE_RE: Lazy<regex::Regex> =
  lazy_regex::lazy_regex!(r"\$\{\{?(\w+)\}?\}");

diff --git a/cli/tests/integration/lsp_tests.rs b/cli/tests/integration/lsp_tests.rs
@@ -7405,7 +7405,7 @@ fn lsp_completions_registry_empty() {
  },
  "newText": "http:https://localhost:4545"
  },
- "commitCharacters": ["\"", "'", "/"]
+ "commitCharacters": ["\"", "'"]
  }]
  })
  );