Merge pull request #97203 from bzarco/launch-multi-root-args

Don't fail when resolving config variables without a `folderUri`.
This commit is contained in:
Andre Weinand
2020-11-18 13:21:00 +01:00
committed by GitHub
4 changed files with 17 additions and 6 deletions

View File

@@ -942,7 +942,7 @@ export class ExtHostVariableResolverService extends AbstractVariableResolverServ
getWorkspaceFolderCount: (): number => {
return folders.length;
},
getConfigurationValue: (folderUri: URI, section: string): string | undefined => {
getConfigurationValue: (folderUri: URI | undefined, section: string): string | undefined => {
return configurationService.getConfiguration(undefined, folderUri).get<string>(section);
},
getExecPath: (): string | undefined => {