David Sancho

Results 217 comments of David Sancho

I tried 0.7.1 with OCaml 5.1, and it doesn't seem to work. ``` { "name": "melange-project", "dependencies": { "ocaml": "5.1.0", "@opam/dune": ">= 3.8.0" }, "devDependencies": { "@opam/ocaml-lsp-server": "*" }, "esy":...

After https://github.com/ml-in-barcelona/quickjs.ml/commit/69d7a0c9675985608f18749c986b3d5a74c3f6b8 and https://github.com/ml-in-barcelona/quickjs.ml/commit/7db97c9812e5e629df6afc72b05661670af55d12 we have the groups but aren't attached to the captures yet neither have an API for reading them

but that's a good tradeoff, no? If you define your own primitive types... it will def break

But why would you define a string type and `string_to_json`/`string_of_json`?

Similar concept/tradeoff from melange-json ppx: https://github.com/melange-community/melange-json/issues/19

It looks like [destructuring](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Destructuring_assignment) isn't supported in quickjs. The workaround is to avoid it, so in case of you not being sure how, I would recommend use a babel repl...

Exactly, the option `output:"json"` is not guaranteed to be always JSON, it just tries to parse as JSON and in case of failing it keeps it as the stdout (which...

Indeed, the empty string is an expected result from jq saying that there's no filters/matches with your query. Either by outputting json or strings.

I like the idea of the undefined, maybe we should have a breaking change and implement this. Are you up for it?

Hey @frekw, you are probably the only one using nix with styled-ppx. If you want to push this forward, I suggest configuring some CI or steps I can run to...