Ulysse

Results 310 comments of Ulysse

> I don't know if this PR can be merged: there are still a few bugs in the extraction of values belonging to local modules that could be improved but...

@Tim-ats-d what do you think of 9bb9388c61cb18b340519f54c7473e58d2acf051 ? This still feels a bit sketchy, but at least more cases are working now.

One option is to wait for ppx_repr to be compatible with ppxlib >= 0.36.0. There is a PR opened: https://github.com/mirage/repr/pull/110 We could try to pin it too.

We are still waiting for the imminent (🌤️ @Julow ☀️) release of OCamlformat 0.28.2 for #1985. Also #2004 does not run the fuzzy CI, so I don't think it's impacted...

So, of course it's linked to shapes of recursive modules: ``` # 0.02 locate - shape_of_path initial: M . "f"[value] # 0.02 locate - shape_of_path reduced: Missing uid # 0.02...

Thanks for your report @daimrod The "output" part of the message doesn't look like the actual error, it is "just" a warning. Could you share the whole log in `~/.opam/log/merlin-65231-dfbc0e.out`...

Is moving the project to OCaml 4.14.X an option for you ? We don't have plans for a 4.13 release soon, but we do support 4.14 more regularly as the...

@Octachron I just rebased this on trunk 🙂 I am sure you are quite busy with the release, let me know if I can do anything to help with the...

I decided to squash all commits when rebasing. I did tests these changes when I wrote the initial version of this PR, but not with the recent changes and they...

> update the checkbox state For reference, my current workaround for that is to sample the `Lwd.var` that holds the state to set the checked property: https://github.com/voodoos/brr-lwd-ui/blob/4d3101122c21dfd2c3d22419d79c2a322bad01c6/lib/forms/field_checkboxes.ml#L81-L87