Skip to content

Use OxCaml#344

Merged
Xyene merged 4 commits intojanestreet:masterfrom
Svetlitski:use-oxcaml
Dec 11, 2025
Merged

Use OxCaml#344
Xyene merged 4 commits intojanestreet:masterfrom
Svetlitski:use-oxcaml

Conversation

@Svetlitski
Copy link
Copy Markdown
Contributor

@Svetlitski Svetlitski commented Dec 8, 2025

Migrate the project to use OxCaml.

I'll upstream the patches to Core_unix and Basement, I've only vendored them for now to unblock things.

@Svetlitski Svetlitski force-pushed the use-oxcaml branch 8 times, most recently from 8f307d8 to 9d3dd8e Compare December 9, 2025 14:47
@Svetlitski Svetlitski marked this pull request as draft December 9, 2025 18:19
Migrate the project from OCaml to [OxCaml](https://oxcaml.org/).

Signed-off-by: Kevin Svetlitski <ksvetlitski@janestreet.com>
Signed-off-by: Kevin Svetlitski <ksvetlitski@janestreet.com>
Signed-off-by: Kevin Svetlitski <ksvetlitski@janestreet.com>
@Svetlitski Svetlitski marked this pull request as ready for review December 11, 2025 04:20
@Svetlitski Svetlitski requested a review from Xyene December 11, 2025 04:20
@Svetlitski Svetlitski changed the title Use oxcaml Use OxCaml Dec 11, 2025
Signed-off-by: Kevin Svetlitski <ksvetlitski@janestreet.com>
Copy link
Copy Markdown
Member

@Xyene Xyene left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@Xyene Xyene merged commit 1d6e208 into janestreet:master Dec 11, 2025
2 checks passed
@Svetlitski Svetlitski deleted the use-oxcaml branch December 11, 2025 20:12
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