mlms13 / bs-decode

Type-safe JSON decoding for ReasonML and OCaml
https://mlms13.github.io/bs-decode/docs/what-and-why
MIT License
103 stars 18 forks source link

Move to melange 2.1 #144

Closed rikimasan closed 9 months ago

rikimasan commented 9 months ago

Hiiii :)

mlms13 commented 9 months ago

The changes look awesome to me! Thanks so much for doing this! Just waiting for CI now, then we can merge this, and I'll tag a a v2.0.0-alpha1 release and we can see about getting this thing onto opam!

mlms13 commented 9 months ago

Ah looks like CI wants OCaml 4 and bs-decode now wants OCaml 5. Would you mind also updating the ci script here to use the latest melange-supported version of OCaml? https://github.com/mlms13/bs-decode/blob/v2/.github/workflows/ci.yml#L18-L19

rikimasan commented 9 months ago

Of course, I think it's just the one line