-
Notifications
You must be signed in to change notification settings - Fork 77
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
Debugger Client in core #804
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.
Looking pretty good, mostly just comment changes here
core/src/debug_client.rs
Outdated
#[derive(Clone)] | ||
pub struct DebugClient { | ||
/// URL for the local instance of the debugger server | ||
debugger_url: String, |
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.
This can be an actual Url
type which we already depend on https://docs.rs/url/latest/url/struct.Url.html (which also has safe ways to construct paths like adding /history
)
Co-authored-by: Spencer Judge <[email protected]>
Co-authored-by: Spencer Judge <[email protected]>
Co-authored-by: Spencer Judge <[email protected]>
Co-authored-by: Spencer Judge <[email protected]>
Closed in favor of #829 |
Adding methods for using our VSCode extension into core, so the debugger can be extended into other languages without having to reimplement these basic methods.
Drafted while I'm still working on it.