Skip to content

Commit 33d46b5

Browse files
committed
src/goDebugConfiguration: fix debugAdapter default selection
The intention of cl/399077 is to choose 'legacy' as the default for 'remote' mode when we are not in preview mode, while choose 'dlv-dap' when we are in preview mode. The condition was reversed. CI couldn't detect the bug because it was running only in preview mode. For #2168 Fixes #2215 Change-Id: I31b353abce29e52270cc589916c61aa5712258e3 Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/402220 Reviewed-by: Polina Sokolova <[email protected]>
1 parent df181c8 commit 33d46b5

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/goDebugConfiguration.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ export class GoDebugConfigurationProvider implements vscode.DebugConfigurationPr
172172
// from package.json (dlv-dap) unless this is remote attach with a stable release.
173173
if (!debugConfiguration['debugAdapter']) {
174174
debugConfiguration['debugAdapter'] = defaultConfig.debugAdapter.default;
175-
if (debugConfiguration['mode'] !== 'remote' && !extensionInfo.isPreview) {
175+
if (debugConfiguration['mode'] === 'remote' && !extensionInfo.isPreview) {
176176
debugConfiguration['debugAdapter'] = 'legacy';
177177
}
178178
}

0 commit comments

Comments
 (0)