search

flycheck / flycheck-ocaml

OCaml support for Flycheck using Merlin
GNU General Public License v3.0
22 stars 6 forks source link

Enable the checker in .merlin presence #16

Open temyurchenko opened 9 months ago

temyurchenko commented 9 months ago

Problem: some projects don't use Dune, while using Merlin, configured with the .merlin file. flycheck-ocaml is only turned on if the «dune-project» file is present though. As a result, flycheck-ocaml is disabled for such projects.

Solution: enable the flycheck-ocaml checker if either dune-project or .merlin are present.

In a round-about way this fixes #15. People who don't use Dune, now don't need to create an empty dune-project to use the checker, which was misleading.

temyurchenko commented 9 months ago

Please, do note that I am not proficient in the OCaml ecosystem and have tested this only on a one-file project. That might entail needing extra attention when reviewing the PR :D