8000 treat Js.import as side-effectful · Issue #1169 · melange-re/melange · GitHub

More Web Proxy on the site http://driver.im/

8000

treat Js.import as side-effectful #1169

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Closed

anmonteiro opened this issue Sep 15, 2024 · 0 comments · Fixed by #1172

Member

anmonteiro commented

•

in #1164, the following snippet:

module type int = (module type of Int);

let () = {
  let _x = Js.import((module Stdlib.Int: int));
  ();
}

produces empty output. We should probably treat Js.import as side-effectful instead because an ignored import should still be imported?

The text was updated successfully, but these errors were encountered:

anmonteiro mentioned this issue

fix: treat dynamic imports as side-effectful #1172

Merged

anmonteiro closed this as completed in #1172

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

0