From 0e7b35da5453a5fbaf6404dbdd549d3cd3223dce Mon Sep 17 00:00:00 2001
From: ckipp01 <ckipp@pm.me>
Date: Tue, 8 Sep 2020 14:34:42 +0200
Subject: [PATCH] Remove isCompletionItemResolve setting.

Without even realizing it, having this in here had been causing
Metals to declare that it wasn't a completionResolve provider
when it reality it was. The reason it wasn't was because I was
telling it to be initialized with the presentation compiler flag
to not provide documentation. I'm assuming this has been in here since
the start, and it was a total oversight that it wasn't removed.

Closes #188
---
 src/index.ts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/index.ts b/src/index.ts
index bab483d..0091c56 100644
--- a/src/index.ts
+++ b/src/index.ts
@@ -181,7 +181,6 @@ async function launchMetals(
   const initializationOptions: MetalsInitializationOptions = {
     compilerOptions: {
       completionCommand: "editor.action.triggerSuggest",
-      isCompletionItemResolve: false,
       overrideDefFormat: "unicode",
       parameterHintsCommand: "editor.action.triggerParameterHints",
     },