hhugo
hhugo
Thanks. Looks good. Can you fix the CI and squash the fixup ?
> Done. It looks like the failing test will only work if I integrate the single commit from #1650, which redefines `Js.number_t` as being `Js.number Js.t`. > > This may...
Turning into a draft until underlying PRs are merged
@OlivierNicole, can you rebase, fix and undraft this PR
Do we really need to pass target around ? It creates a lot of noise for few uses only. It could be turned into an global config like `use_js_string` or...
Looking at the diff, I find it hard to extract where the behavior depend on the target because there are so many targets passed around. I like that values are...
> Wouldn’t a global flag make it harder to use the jsoo compiler as a library, which could be invoked many times with different targets? Sure, but we already have...
@OlivierNicole, the CI is unhappy, and the diff has extra lines because of formatting. Can you run `dune fmt` ?
What's the semantics of nativeInt with the wasm backend?
Then we should use | NativeInt of Int32.t