Skip to content
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.

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

fix(melange): pass melange.emit's compile_flags to js emission #11252

Merged

Conversation

anmonteiro
Copy link
Collaborator

No description provided.

@anmonteiro anmonteiro added the melange Melange rules and generator label Dec 30, 2024
@anmonteiro anmonteiro changed the title test(melange): show that .js file emission doesn't respect flags fix(melange): pass melange.emit's compile_flags to js emission Dec 30, 2024
@anmonteiro anmonteiro force-pushed the anmonteiro/melange-emision-compile-flags branch from 8b4c5fe to 7cec72b Compare January 1, 2025 02:40
Signed-off-by: Antonio Nuno Monteiro <[email protected]>
@anmonteiro anmonteiro merged commit 8a30fb1 into ocaml:main Jan 1, 2025
1 of 2 checks passed
@anmonteiro anmonteiro deleted the anmonteiro/melange-emision-compile-flags branch January 1, 2025 03:01
@anmonteiro anmonteiro mentioned this pull request Jan 21, 2025
7 tasks
anmonteiro added a commit to anmonteiro/dune that referenced this pull request Jan 22, 2025
…caml#11252)

* test(melange): show that .js file emission doesn't respect flags

Signed-off-by: Antonio Nuno Monteiro <[email protected]>

* fix(melange): pass `melange.emit`'s `compile_flags` to js emission

Signed-off-by: Antonio Nuno Monteiro <[email protected]>

* chore: add changelog entry

Signed-off-by: Antonio Nuno Monteiro <[email protected]>

---------

Signed-off-by: Antonio Nuno Monteiro <[email protected]>
maiste pushed a commit that referenced this pull request Jan 23, 2025
…to js emission (#11370)

* test(melange): show that .js file emission doesn't respect flags
* fix(melange): pass `melange.emit`'s `compile_flags` to js emission
* chore: add changelog entry
---------
Signed-off-by: Antonio Nuno Monteiro <[email protected]>
ElectreAAS pushed a commit to ElectreAAS/dune that referenced this pull request Jan 27, 2025
…caml#11252)

* test(melange): show that .js file emission doesn't respect flags

Signed-off-by: Antonio Nuno Monteiro <[email protected]>

* fix(melange): pass `melange.emit`'s `compile_flags` to js emission

Signed-off-by: Antonio Nuno Monteiro <[email protected]>

* chore: add changelog entry

Signed-off-by: Antonio Nuno Monteiro <[email protected]>

---------

Signed-off-by: Antonio Nuno Monteiro <[email protected]>
chris-armstrong pushed a commit to chris-armstrong/dune that referenced this pull request Jan 29, 2025
…caml#11252)

* test(melange): show that .js file emission doesn't respect flags

Signed-off-by: Antonio Nuno Monteiro <[email protected]>

* fix(melange): pass `melange.emit`'s `compile_flags` to js emission

Signed-off-by: Antonio Nuno Monteiro <[email protected]>

* chore: add changelog entry

Signed-off-by: Antonio Nuno Monteiro <[email protected]>

---------

Signed-off-by: Antonio Nuno Monteiro <[email protected]>
Signed-off-by: Chris Armstrong <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
melange Melange rules and generator
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants