Skip to content

Conversation

@xvw
Copy link
Collaborator

@xvw xvw commented Jan 20, 2026

For ocaml-eglot, it make sense to relay on a custom request instead of a code action for punctually trigger the command.

@xvw xvw force-pushed the destruct-code-action branch from e13b017 to 4f7e3c8 Compare January 20, 2026 13:36
@xvw xvw requested a review from voodoos January 20, 2026 13:36
@xvw xvw changed the title Destruct code action Destruct custom request Jan 21, 2026
Copy link
Collaborator

@voodoos voodoos left a comment

Choose a reason for hiding this comment

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

LGTM

@voodoos
Copy link
Collaborator

voodoos commented Jan 28, 2026

Thanks !

CHANGES.md Outdated
- Add `typeExpression` custom request (#1576)
- Add `locate` custom request (#1576)
- Add `phrase` custom request (#1576)
- Add `destruct` custom request (#1583)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ah ! But 1.25.0 was released already, we need a new changelog section :-)
Next time we should add the section right after the release...

@voodoos voodoos merged commit 1799dcf into ocaml:master Jan 28, 2026
4 of 6 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.

2 participants