Skip to content

Conversation

hhugo
Copy link
Contributor

@hhugo hhugo commented Jul 7, 2025

Compiler from OCaml bytecode to JavaScript

CHANGES:

Bug fixes

CHANGES:

## Bug fixes
* Compiler: Fix shape loading (ocsigen/js_of_ocaml#2074)
@hhugo
Copy link
Contributor Author

hhugo commented Jul 7, 2025

same errors as #28135

@hhugo
Copy link
Contributor Author

hhugo commented Jul 9, 2025

same errors as #28135

Just to clarify, errors are unrelated, this is good to go.

@mseri
Copy link
Member

mseri commented Jul 10, 2025

The gen_js_api seems due to a change in output. Are you sure we should not add an upper bound on the tests?

@hhugo
Copy link
Contributor Author

hhugo commented Jul 10, 2025

The gen_js_api seems due to a change in output. Are you sure we should not add an upper bound on the tests?

We could probably disable the tests for problematic combination of ppxlib+gen_js_api. But id prefer not to delay this PR too much.

@mseri
Copy link
Member

mseri commented Jul 10, 2025

I had the PR ready :)

Thanks!

@mseri mseri merged commit cd1d2da into ocaml:master Jul 10, 2025
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants