Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

debug: find a way to use dlv-dap with debug test codelens #1293

Closed
hyangah opened this issue Mar 11, 2021 · 3 comments
Closed

debug: find a way to use dlv-dap with debug test codelens #1293

hyangah opened this issue Mar 11, 2021 · 3 comments
Assignees
@suzmue
Labels
Debug Issues related to the debugging functionality of the extension. FrozenDueToAge
Projects
Enable Delve DAP
Milestone
v0.24.1

Comments

@hyangah
Copy link
Contributor

hyangah commented Mar 11, 2021

Due to the issue #855
we currently don't have an easy way to run debugging using dlv-dap.
Either we need to fix the issue, or take advantage of "go.delveConfig" setting.

cc @polinasok @suzmue
@hyangah hyangah added this to Needs triage in Enable Delve DAP via automation Mar 11, 2021
@hyangah hyangah changed the title debug: find a way to use dlv-dap with debug test codelense debug: find a way to use dlv-dap with debug test codelens Mar 15, 2021
@hyangah hyangah moved this from Needs triage to Low priority in Enable Delve DAP Mar 25, 2021
@hyangah hyangah added Debug Issues related to the debugging functionality of the extension. DlvDAP labels Mar 25, 2021
@gopherbot
Copy link
Collaborator

Change https://golang.org/cl/306969 mentions this issue: src/goTest.ts: allow users to select dlv-dap in codelenses

@gopherbot gopherbot added this to the Untriaged milestone Apr 8, 2021
@hyangah hyangah moved this from Low priority to In Progress in Enable Delve DAP Apr 12, 2021
Enable Delve DAP automation moved this from In Progress to Closed Apr 12, 2021
@hyangah hyangah modified the milestones: Untriaged, v0.24.1 Apr 13, 2021
@hyangah
Copy link
Contributor Author

hyangah commented Apr 13, 2021

@suzmue I think we need documentation update.

@gopherbot
Copy link
Collaborator

Change https://golang.org/cl/309749 mentions this issue: docs/dlv-dap.md: include go.delveConfig setting for debugAdapter

gopherbot pushed a commit that referenced this issue Apr 13, 2021
@suzmue
docs/dlv-dap.md: include go.delveConfig setting for debugAdapter
6f9c0eb 
With the addition of the "debugAdapter" attribute to the
go.delveConfig setting, users can easily select dlv-dap for
all debug sessions including codelenses.

Updates #1293

Change-Id: Icc4cedf3e684418069911728cfb66efec6662167
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/309749
Trust: Suzy Mueller <suzmue@golang.org>
Run-TryBot: Suzy Mueller <suzmue@golang.org>
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
@golang golang locked and limited conversation to collaborators Apr 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@suzmue suzmue

Labels
Debug Issues related to the debugging functionality of the extension. FrozenDueToAge
Projects
No open projects
Enable Delve DAP
Closed
Milestone
v0.24.1
Development

No branches or pull requests
3 participants
@hyangah @suzmue @gopherbot