Skip to content

[new release] lsp (3 packages) (1.25.0) - #29118

Merged
jmid merged 2 commits into
ocaml:masterfrom
voodoos:release-lsp-1.25.0
Dec 20, 2025
Merged

[new release] lsp (3 packages) (1.25.0)#29118
jmid merged 2 commits into
ocaml:masterfrom
voodoos:release-lsp-1.25.0

Conversation

@voodoos

@voodoos voodoos commented Dec 20, 2025

Copy link
Copy Markdown
Contributor

LSP protocol implementation in OCaml

CHANGES:

Features

Fixes

CHANGES:

## Features

- Make `code-lens` for nested let bindings configurable (ocaml/ocaml-lsp#1567)
- Add support for `.mlx` files, including formatting via `ocamlformat-mlx` and most OCaml LSP features (diagnostics, code actions, hover, etc.) (ocaml/ocaml-lsp#1528)
- Add `typeExpression` custom request (ocaml/ocaml-lsp#1576)
- Add `locate` custom request (ocaml/ocaml-lsp#1576)
- Add `phrase` custom request (ocaml/ocaml-lsp#1576)

## Fixes

- Improve precision of collected metrics timestamps. (ocaml/ocaml-lsp#1565)
- Fallback on `.merlin` configuration if no `dune-project` file is found and if
  `dot-merlin-reader` is installed. (ocaml/ocaml-lsp#1563, fixes ocaml/ocaml-lsp#1522)

@jmid jmid left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

There are lower bound yojson failures.

I can see you earlier added a lower bound for it in 8b437ad from #28650, which would be good to add upstream 🙂

@voodoos

voodoos commented Dec 20, 2025

Copy link
Copy Markdown
Contributor Author

Sorry for that. I fixed it upstream too.

@jmid jmid left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, this now LGTM!

CI is down to a single opam-2.0 failure.

@jmid
jmid merged commit a2ea42d into ocaml:master Dec 20, 2025
4 of 5 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.

Why isn't --fallback-read-dot-merlin the default?

2 participants