Skip to content
This repository was archived by the owner on Nov 18, 2022. It is now read-only.

Commit 3a6466d

Browse files
committed
apply hack only to windows + add clarifying comment
1 parent 1a761d8 commit 3a6466d

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

src/extension.ts

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -207,8 +207,15 @@ class ClientWorkspace {
207207
return this.makeRlsProcess();
208208
};
209209

210+
211+
// Something else is put front of files when pattern matching that prevents the windows version from picking up the files
212+
// This should be safe as the uri is a absolute path that includes the drive + colon
213+
// i.e. a pattern would become "**/c:/some/path**" and since colon is reserved only the root can ever contain it.
214+
const isWin = process.platform === "win32";
215+
const windowsHack = isWin ? "**" : "";
216+
210217
const pattern = this.config.multiProjectEnabled
211-
? `**${this.folder.uri.path}/**`
218+
? `${windowsHack}${this.folder.uri.path}/**`
212219
: undefined;
213220

214221
const collectionName = this.config.multiProjectEnabled

0 commit comments

Comments
 (0)