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

Misplaced error when .merlin file not found #333

Closed
yannl35133 opened this issue Dec 4, 2020 · 3 comments
Closed

Misplaced error when .merlin file not found #333

yannl35133 opened this issue Dec 4, 2020 · 3 comments

Comments

@yannl35133
Copy link

I recently found out the error message : No configuration file found. Try calling `dune build` to generate `.merlin` files. ocamllsp

This is a very nice addition, but it is placed at line 2 column 1, which seems odd : why not place it at line 1?
Could you link to the PR which added this, I cannot find it (it is maybe older than I think) ?

Also, two little remarks which are not really worthy of an issue : in CHANGES.md, version 1.2.0, Fixes,

#245: correctly use mutexes on OpenBSD (#264)

(Why no capital letter for Correctly?)

Disable non-prefix completion. There's no reliably way to trigger it and it can be slow.

(reliable instead of reliably, no?)

@rgrinberg
Copy link
Member

I recently found out the error message : No configuration file found. Try calling dune build to generate .merlin files. ocamllsp

Which versions of lsp and dune are you using?

This is a very nice addition, but it is placed at line 2 column 1, which seems odd : why not place it at line 1?
Could you link to the PR which added this, I cannot find it (it is maybe older than I think) ?

That's because the change was done upstream in merlin.

Also, two little remarks which are not really worthy of an issue : in CHANGES.md, version 1.2.0, Fixes,

Patches are welcome.

@yannl35133
Copy link
Author

Which versions of lsp and dune are you using?

I'm using the latest for both, they are both pinned :

$ dune-dev --version  # An alias for dune on another switch, the one that I use
2.7.0-370-g5a2e49be6
$ ocamllsp --version
1.3.0-38-g23546b370

That's because the change was done upstream in merlin.

Oh right

Patches are welcome.

I'm not sure to understand

@rgrinberg
Copy link
Member

This is no longer relevant. Please re-open if there's still a problem.

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

No branches or pull requests

2 participants