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

Sync DAP interfaces and types with DAP 1.69.x spec #863

Merged
merged 10 commits into from
Jan 13, 2025

Conversation

jansorg
Copy link
Contributor

@jansorg jansorg commented Jan 7, 2025

Closes #862

This syncs LSP4J's data types and interfaces with the latest version of the DAP specification, version 1.69.
I kept the commit messages the same as the items in the DAP changelog at https://microsoft.github.io/debug-adapter-protocol/changelog

@jansorg jansorg force-pushed the dap-spec-sync branch 5 times, most recently from 3b031ef to b51afd4 Compare January 7, 2025 21:06
@jansorg jansorg force-pushed the dap-spec-sync branch 2 times, most recently from b28b343 to 17165d0 Compare January 8, 2025 09:18
@jansorg jansorg marked this pull request as ready for review January 8, 2025 09:18
@jonahgraham jonahgraham requested a review from KamasamaK January 8, 2025 17:39
@jonahgraham
Copy link
Contributor

Copy link
Contributor

@jonahgraham jonahgraham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jansorg for doing all the heavy lifting here on bringing this up to 1.69.

I added a few additional documentation changes and the missing supportsDataBreakpointBytes in Capabilities.

Can you have a look at the changes I just pushed to ensure that I didn't undo anything you intentionally did.

Thanks again!

@jansorg
Copy link
Contributor Author

jansorg commented Jan 13, 2025

@jonahgraham Thanks for the review! Your changes seem fine to me.

@jonahgraham jonahgraham merged commit 651eb49 into eclipse-lsp4j:main Jan 13, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DAP: Variable.declarationLocationReference and Variable.valueLocationReference are missing
2 participants