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

Added parentheses in Dyn.pp output. #10864

Merged
merged 1 commit into from
Aug 30, 2024
Merged

Conversation

tov
Copy link
Collaborator

@tov tov commented Aug 30, 2024

Teaches Dyn.pp to parenthesize constructor applications properly.

Signed-off-by: Jesse Tov <jtov@janestreet.com>
@tov tov force-pushed the add-parents-to-Dyn.pp-output branch from 22dd0b7 to dafcc72 Compare August 30, 2024 13:21
@rgrinberg rgrinberg merged commit c82db6a into main Aug 30, 2024
27 of 28 checks passed
@rgrinberg rgrinberg deleted the add-parents-to-Dyn.pp-output branch August 30, 2024 14:02
emillon added a commit to emillon/dune that referenced this pull request Sep 2, 2024
It has been changed in ocaml#10864.

Signed-off-by: Etienne Millon <me@emillon.org>
emillon added a commit that referenced this pull request Sep 2, 2024
It has been changed in #10864.

Signed-off-by: Etienne Millon <me@emillon.org>
anmonteiro pushed a commit to anmonteiro/dune that referenced this pull request Nov 17, 2024
Signed-off-by: Jesse Tov <jtov@janestreet.com>
anmonteiro pushed a commit to anmonteiro/dune that referenced this pull request Nov 17, 2024
It has been changed in ocaml#10864.

Signed-off-by: Etienne Millon <me@emillon.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants