Skip to content

chore: update vendored pp to 2.0.0#10938

Merged
Leonidas-from-XIV merged 1 commit intoocaml:mainfrom
gridbugs:upgrade-pp.2.0.0
Sep 20, 2024
Merged

chore: update vendored pp to 2.0.0#10938
Leonidas-from-XIV merged 1 commit intoocaml:mainfrom
gridbugs:upgrade-pp.2.0.0

Conversation

@gridbugs
Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: Stephen Sherratt <stephen@sherra.tt>
@gridbugs
Copy link
Copy Markdown
Collaborator Author

Fixes #10934

Copy link
Copy Markdown
Collaborator

@Leonidas-from-XIV Leonidas-from-XIV left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks! I like how removing the formatting constructor allowed us to get rid of some dodgy conversion code.

@Leonidas-from-XIV Leonidas-from-XIV merged commit 4c28fa0 into ocaml:main Sep 20, 2024
@Leonidas-from-XIV
Copy link
Copy Markdown
Collaborator

Merging since this fixes the annoying CI failure and I can rebase my PRs on top of this.

anmonteiro pushed a commit to anmonteiro/dune that referenced this pull request Nov 17, 2024
Signed-off-by: Stephen Sherratt <stephen@sherra.tt>
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