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
dot-merlin-reader: init at 3.4.0 #98603
Conversation
10a5975
to
df0d91c
Compare
cc @vbgl |
df0d91c
to
8ebef6c
Compare
Does this need to be part of If not, please move it out of that set. This will make it easier to use dot-merlin-reader with OCaml < 4.08. |
@vbgl that's a good point, I don't think the OCaml version matters. AFAICT the sole responsibility of The only requirement is that the version of |
I fear that the opam description does not tell you about this kind of constraints. The |
Took a look at the protocol, it uses |
Thanks for investigating. Then, please do move |
8ebef6c
to
147018f
Compare
Done and updated title. |
@GrahamcOfBorg build dot-merlin-reader |
Motivation for this change
dot-merlin-reader
is a new dependency formerlin
3.4.0 (which I plan to do in a separate PR). It serves as an external config provider formerlin
. The intention is that future versions ofmerlin
will be able to source config from different providers (e.g.,dune
).Reference PRs:
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)