-
Notifications
You must be signed in to change notification settings - Fork 287
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
Added debugger key in kernel_info_reply #486
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @JohanMabille! This will be useful for clients who want to offer debugging capabilities only when a kernel supports debugging.
My approval does not mean "ready to merge" as much as it means 👍 I know there are a multiple stakeholders here who might have thoughts on this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @JohanMabille - just had the one (minor) comment.
b1fe8e4
to
3e8ee4a
Compare
@JohanMabille I've added this to the 7.0 milestone, can you please rebase? |
@blink1073 sure |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
This is adding the 'debugger' key toe the
kernel_info_reply
message to signal the frontent if it can start a debugging session.We may map a more complicated structure to this key, to specify subparts of the Debug Adapter protocol that are supported for instance (or any additional requirement).