Skip to content

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

Merged
jonahgraham merged 10 commits into
eclipse-lsp4j:mainfrom
jansorg:dap-spec-sync
Jan 13, 2025
Merged

Sync DAP interfaces and types with DAP 1.69.x spec#863
jonahgraham merged 10 commits into
eclipse-lsp4j:mainfrom
jansorg:dap-spec-sync

Conversation

@jansorg

@jansorg jansorg commented Jan 7, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor

@jonahgraham jonahgraham left a comment

Copy link
Copy Markdown
Contributor

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

jansorg commented Jan 13, 2025

Copy link
Copy Markdown
Contributor Author

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

@jonahgraham jonahgraham merged commit 651eb49 into eclipse-lsp4j:main Jan 13, 2025
@jonahgraham jonahgraham added this to the 0.24.0 milestone Jan 31, 2025
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