Skip to content

Commit

Permalink
debug: fix dynamic configurations in empty workspace (microsoft#232520)
Browse files Browse the repository at this point in the history
We would skip trying to find configurations instead of passing an undefined workspace folder (which is entirely legal)

Fixes microsoft#228949
  • Loading branch information
connor4312 authored Oct 29, 2024
1 parent 22b0035 commit f992298
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -226,8 +226,8 @@ export class ConfigurationManager implements IConfigurationManager {
const picks: Promise<IDynamicPickItem[]>[] = [];
const provider = this.configProviders.find(p => p.type === type && p.triggerKind === DebugConfigurationProviderTriggerKind.Dynamic && p.provideDebugConfigurations);
this.getLaunches().forEach(launch => {
if (launch.workspace && provider) {
picks.push(provider.provideDebugConfigurations!(launch.workspace.uri, token.token).then(configurations => configurations.map(config => ({
if (provider) {
picks.push(provider.provideDebugConfigurations!(launch.workspace?.uri, token.token).then(configurations => configurations.map(config => ({
label: config.name,
description: launch.name,
config,
Expand Down

0 comments on commit f992298

Please sign in to comment.