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

feat(dap): implement Restart request #5651

Merged

Conversation

filipdutescu
Copy link
Contributor

Add a restart debug session command, which would issue a Restart Request, if the debugger supports it and a session is running. It uses the same arguments and requests used to start the initial session, when recreating it.

It builds upon #5532, making use of the changes to the termination workflow of a session.

Closes: #5594
Signed-off-by: Filip Dutescu [email protected]

@the-mikedavis the-mikedavis added C-enhancement Category: Improvements S-waiting-on-review Status: Awaiting review from a maintainer. A-debug-adapter Area: Debug adapter client labels Jan 23, 2023
@archseer
Copy link
Member

This needs a rebase now

Add a restart debug session command, which would issue a
[Restart Request][1], if the debugger supports it and a session is
running. It uses the same arguments and requests used to start the
initial session, when recreating it.

It builds upon helix-editor#5532, making use of the changes to the termination
workflow of a session.

[1]: https://microsoft.github.io/debug-adapter-protocol/specification#Requests_Restart

Closes: helix-editor#5594
Signed-off-by: Filip Dutescu <[email protected]>
@filipdutescu
Copy link
Contributor Author

Should all be good now

@archseer archseer merged commit 376c19e into helix-editor:master Mar 6, 2023
sagnibak pushed a commit to sagnibak/helix that referenced this pull request Mar 21, 2023
Add a restart debug session command, which would issue a
[Restart Request][1], if the debugger supports it and a session is
running. It uses the same arguments and requests used to start the
initial session, when recreating it.

It builds upon helix-editor#5532, making use of the changes to the termination
workflow of a session.

[1]: https://microsoft.github.io/debug-adapter-protocol/specification#Requests_Restart

Closes: helix-editor#5594

Signed-off-by: Filip Dutescu <[email protected]>
wes-adams pushed a commit to wes-adams/helix that referenced this pull request Jul 4, 2023
Add a restart debug session command, which would issue a
[Restart Request][1], if the debugger supports it and a session is
running. It uses the same arguments and requests used to start the
initial session, when recreating it.

It builds upon helix-editor#5532, making use of the changes to the termination
workflow of a session.

[1]: https://microsoft.github.io/debug-adapter-protocol/specification#Requests_Restart

Closes: helix-editor#5594

Signed-off-by: Filip Dutescu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-debug-adapter Area: Debug adapter client C-enhancement Category: Improvements S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add restart debug session option, if the DAP adapter supports it.
3 participants