Nicolas Berthier

This seems related to the changes introduced from `ppx_tools_versioned.5.2` to `ppx_tools_versioned.5.2.1` (see https://github.com/mirage/ocaml-cstruct/pull/204 for more info on why there is now a need for explicitly stating runtime dependencies on `ppx_tools_versions`).

Ok thanks that's a way. However note that with the dune trailer added as you suggested along with a `generators = [… "menhir"]`, the generated `dune` file has multiple rules...

Yes indeed. I'll leave the issue open as a reminder.

Yes I understand about the quest for having very few dependencies. My use-case, is an attempt at re-using the recovery algorithm as instantiated in `Merlin_kernel.Mreader_recover` with a grammar for another...

@LilyOlivier Out of curiosity, are comma separators (`,` followed by any number of blank/space characters) relevant in SQL? Is yes, I might need to apply some non-trivial changes to the...

> @nberth I think they are, […] I hope this will answer your question! Yes thanks. Can we encounter semicolons `;` as well?

Once #279 is merged, you should be able to handle those `,` (they will appear as `Separator ','` in the list of text words — of type `text`).

> I'd say we should merge this ; it will be easier to move on to the next tasks. What do you think @nberth ? (I'm currently at the Frama-C...

> Maybe i should squash commit first? Yes indeed. If that's into a single commit, then that can be done here. Otherwise, I let you do a squash as you...

> This PR is on hold until the next release of [ocaml-lsp](https://github.com/ocaml/ocaml-lsp) `>1.17.0`, which is needed for a bug fix. [Looks like](https://github.com/ocaml/opam-repository/pull/26202) it's imminent ;-)