Explicitly return undefined from resolveDbgConfig when sessn not started (#1548)

rkeithhill · rjmholt · commit 7afec4445383 · 2018-09-27T10:16:39.000-07:00
In the September drop of VSCode this fixes the issue with VSCode opening launch.json in this case. Technically just returning nothing works but better to be explicit in this case I think. microsoft/vscode#54213 (comment)
diff --git a/src/features/DebugSession.ts b/src/features/DebugSession.ts
@@ -53,7 +53,7 @@ export class DebugSessionFeature implements IFeature, DebugConfigurationProvider
             const msg = "Cannot debug or run a PowerShell script until the PowerShell session has started. " +
                 "Wait for the PowerShell session to finish starting and try again.";
             vscode.window.showWarningMessage(msg);
-            return;
+            return undefined;
         }
 
         // Starting a debug session can be done when there is no document open e.g. attach to PS host process

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ export class DebugSessionFeature implements IFeature, DebugConfigurationProvider`
`53`	`53`	`const msg = "Cannot debug or run a PowerShell script until the PowerShell session has started. " +`
`54`	`54`	`"Wait for the PowerShell session to finish starting and try again.";`
`55`	`55`	`vscode.window.showWarningMessage(msg);`
`56`		`- return;`
	`56`	`+ return undefined;`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`// Starting a debug session can be done when there is no document open e.g. attach to PS host process`