jaredly / jaredly/reason-language-server

rls gets confused by stray `.merlin` files

Open
#225 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
OCaml
Stars
649
Forks
85
PR merge metrics
No merged PRs in 30d

Description

See https://reasonml.chat/t/how-to-debug-reason-vscode-and-an-error-could-not-find-local-build-dir/1370

Currently it uses the presence of a `.merlin` file in a source directory as an indicator that we're dealing with a dune project, but we should probably ignore that b/c the presence of a `dune` file is the real indicator.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the project-detection entry point that treats a .merlin file in a source directory as evidence of a dune project. Reproduce the stray-.merlin case described in the linked discussion, then make detection rely on the presence of a dune file and verify that genuine dune projects still work.

Written by the indexing model from the issue text.

Assessment

Tech stack
ocaml
Domain
devtools
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.