search

marigold-dev / pollinate

A platform agnostic library for P2P communications using UDP and Bin_prot
22 stars 2 forks source link

ci(ocaml5): get rid of ocaml 4.08 and add ocaml 5 #29

Closed Gau-thier closed 2 years ago

Gau-thier commented 2 years ago

Depends

rosalogia commented 2 years ago

I don't know much about CI, so this looks okay to me, but it's probably best if we have @ulrikstrid weigh in if you're unsure about some things.

Gau-thier commented 2 years ago

@rosalogia @ulrikstrid should I keep the 4.13 also? I think it is useless since I added the esy workflow, which will test the 4.13 since we have it within package.json

  "dependencies": {
    "ocaml": "4.13.x"
    [...]
  }

What do you think?

ulrikstrid commented 2 years ago

@gsebil08 Since the goal is to use this via deku I would rather have ocaml 5 in esy and then test other versions in opam