Skip to content

[new release] starred_ml (0.0.8)#29551

Merged
mseri merged 2 commits intoocaml:masterfrom
paulosuzart:release-starred_ml-0.0.8
Mar 17, 2026
Merged

[new release] starred_ml (0.0.8)#29551
mseri merged 2 commits intoocaml:masterfrom
paulosuzart:release-starred_ml-0.0.8

Conversation

@paulosuzart
Copy link
Contributor

Generates a awesome list markdown

CHANGES:
  • Replaced deprecated Mirage_crypto_rng_eio.run with Mirage_crypto_rng_unix.use_default from mirage-crypto-rng.unix.
  • Improved error handling: HTTP errors (4xx/5xx) print a clean message; unexpected failures show a full stack trace.
  • Bumped dependencies: eio and eio_main >= 1.3, cohttp-eio >= 6.2.1, mirage-crypto-rng >= 1.2.0, alcotest >= 1.9.1, dune lang 3.21.
  • Updated CI to OCaml 5.4.1.
  • Updated README with development setup instructions.
  • Updated OCaml setup GitHub Actions version.

CHANGES:

- Replaced deprecated `Mirage_crypto_rng_eio.run` with `Mirage_crypto_rng_unix.use_default` from `mirage-crypto-rng.unix`.
- Improved error handling: HTTP errors (4xx/5xx) print a clean message; unexpected failures show a full stack trace.
- Bumped dependencies: `eio` and `eio_main` >= 1.3, `cohttp-eio` >= 6.2.1, `mirage-crypto-rng` >= 1.2.0, `alcotest` >= 1.9.1, `dune` lang 3.21.
- Updated CI to OCaml 5.4.1.
- Updated README with development setup instructions.
- Updated OCaml setup GitHub Actions version.
@paulosuzart paulosuzart marked this pull request as ready for review March 17, 2026 00:20
@paulosuzart
Copy link
Contributor Author

paulosuzart commented Mar 17, 2026

Hello folks, there's a Windows target failing. Not really sure about the best approach, maybe using "ocaml" {>= "5.2.0"} at starred_ml.opam would do it? TIA.

@mseri
Copy link
Member

mseri commented Mar 17, 2026

Windows failure is independent of this PR:


  #=== ERROR while compiling core.v0.16.2 =======================================#
  # context     2.5.0 | win32/x86_64 | ocaml.5.2.1 | file://D:/a/opam-repository/opam-repository
  # path        D:\opamroot\default\.opam-switch\build\core.v0.16.2
  # command     D:\opamroot\default\bin\dune.exe build -p core -j 3
  # exit-code   1
  # env-file    D:\opamroot\log\core-1236-6d160d.env
  # output-file D:\opamroot\log\core-1236-6d160d.out
  ### output ###
  # D:/opamroot/default/lib/ocaml/caml/major_gc.h:62:16: warning: 'caml_major_cycles_completed' redeclared without dllimport attribute: previous dllimport ignored [-Wattributes]
  # [...]
  #       |                ^~~~~~~~~~~~~~~~~~~~~~~~~~~
  # (cd _build/default && D:\opamroot\default\bin\ocamlc.opt.exe -w -40 -g -bin-annot -bin-annot-occurrences -I core/src/.core.objs/byte -I D:\opamroot\default\lib\base -I D:\opamroot\default\lib\base\base_internalhash_types -I D:\opamroot\default\lib\base\caml -I D:\opamroot\default\lib\base\md5 -I D:\opamroot\default\lib\base\shadow_stdlib -I D:\opamroot\default\lib\base_bigstring -I D:\opamroo[...]
  # File "core/src/gc.ml", line 507, characters 4-54:
  # 507 |     Stdlib.Gc.Memprof.start ~sampling_rate:1.0 tracker;
  #           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  # Warning 10 [non-unit-statement]: this expression should have type unit.
  # (cd _build/default && D:\opamroot\default\bin\ocamlopt.opt.exe -w -40 -g -I core/src/.core.objs/byte -I core/src/.core.objs/native -I D:\opamroot\default\lib\base -I D:\opamroot\default\lib\base\base_internalhash_types -I D:\opamroot\default\lib\base\caml -I D:\opamroot\default\lib\base\md5 -I D:\opamroot\default\lib\base\shadow_stdlib -I D:\opamroot\default\lib\base_bigstring -I D:\opamroot\[...]
  # File "core/src/gc.ml", line 507, characters 4-54:
  # 507 |     Stdlib.Gc.Memprof.start ~sampling_rate:1.0 tracker;
  #           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  # Warning 10 [non-unit-statement]: this expression should have type unit.

Good one. Thanks

Co-authored-by: Marcello Seri <mseri@users.noreply.github.com>
@paulosuzart paulosuzart requested a review from mseri March 17, 2026 18:11
@mseri mseri merged commit f2c5c43 into ocaml:master Mar 17, 2026
3 of 5 checks passed
@mseri
Copy link
Member

mseri commented Mar 17, 2026

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants