Conversation

andyleejordan

Resolves #2153.

@andyleejordanandyleejordan force-pushed the andschwa/multi-workspace branch from 5aa078b to 710f6d8 Compare December 1, 2022 21:01
@andyleejordanandyleejordan changed the title WIP: Starting to support multi-root workspaces Fully support multi-root workspaces Dec 1, 2022
@andyleejordanandyleejordan marked this pull request as ready for review December 1, 2022 21:02
@andyleejordanandyleejordan requested a review from a team December 1, 2022 21:02
@ghost ghost added Area-Workspaces Issue-EnhancementA feature request (enhancement).labels Dec 1, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@andyleejordanandyleejordan force-pushed the andschwa/multi-workspace branch from 710f6d8 to 46e0e77 Compare December 1, 2022 21:34
@@ -24,6 +24,9 @@ export function getPipePath(pipeName: string) {
// Check that the file or directory exists in an asynchronous manner that relies
// solely on the VS Code API, not Node's fs library, ignoring symlinks.
async function checkIfFileOrDirectoryExists(targetPath: string | vscode.Uri, type: vscode.FileType): Promise<boolean> {
if (targetPath === "") {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SeeminglyScience since the settings overhaul PR, the default cwd was "" which conspicuously vscode.workspace.fs.stat was totally fine saying it existed.

@andyleejordanandyleejordan enabled auto-merge (squash) December 1, 2022 21:36
@andyleejordanandyleejordan merged commit 6152c60 into main Dec 1, 2022
@andyleejordanandyleejordan deleted the andschwa/multi-workspace branch December 1, 2022 21:44
@andyleejordanandyleejordan restored the andschwa/multi-workspace branch December 1, 2022 21:47
@andyleejordanandyleejordan deleted the andschwa/multi-workspace branch December 1, 2022 21:47
Sign up for free to join this conversation on . Already have an account? Sign in to comment
Area-Workspaces Issue-EnhancementA feature request (enhancement).
None yet

Successfully merging this pull request may close these issues.

Avoid using workspace.rootPath