Skip to content

Upgrade OCaml version requirements, build system, and enable windows CI#10

Open
nberth wants to merge 1 commit into
OCamlPro:masterfrom
nberth:drom-upgrade
Open

Upgrade OCaml version requirements, build system, and enable windows CI#10
nberth wants to merge 1 commit into
OCamlPro:masterfrom
nberth:drom-upgrade

Conversation

@nberth

@nberth nberth commented Jun 12, 2026

Copy link
Copy Markdown
  • Use drom.0.9.3 & drom-share.0.9.3
  • Increase version to 0.4.1
  • Use OCaml.4.12.0 edition (min available on macos)
  • Use OCaml.4.11.0 min-edition (min available in opam repository)
  • Remove dependency on ocmalformat
  • Enable Windows CI

Subsumes #8 .

@nberth nberth force-pushed the drom-upgrade branch 3 times, most recently from 949a5be to 6173517 Compare June 12, 2026 08:53
@lefessan

Copy link
Copy Markdown
Member

Pour moi, la min-editiion ne doit pas dépendre de la disponibilité sur une plateforme, mais de la compatibilité du code, non ?

@nberth

nberth commented Jun 12, 2026

Copy link
Copy Markdown
Author

Pour moi, la min-editiion ne doit pas dépendre de la disponibilité sur une plateforme, mais de la compatibilité du code, non ?

Ah oui c'est probable en effet. Sauf qu'il faut que cette version soit encore disponible dans le dépôt OCaml (ce qui nous amène à 4.11.0 je crois).

@nberth nberth force-pushed the drom-upgrade branch 2 times, most recently from 3ca2b74 to ed53a28 Compare June 12, 2026 10:21
Additional:
- Increase version to 0.4.1
- Use OCaml.4.14.2 edition (min available on windows)
- Use OCaml.4.11.0 min-edition (min available in opam repository)
- Remove dependency on `ocamlformat`
- Enable Windows CI
@nberth nberth changed the title Use drom.0.9.3 & drom-share.0.9.3 Upgrade OCaml, build system, and enable windows CI Jun 12, 2026
@nberth nberth changed the title Upgrade OCaml, build system, and enable windows CI Upgrade OCaml version requirements, build system, and enable windows CI Jun 12, 2026
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