ocurrent / ocaml-ci

A CI for OCaml projects
https://ocaml.ci.dev
112 stars 74 forks source link

Move lower-bound to `Vars.t` and `Selection.t` #824

Closed benmandrew closed 1 year ago

benmandrew commented 1 year ago

Relies on https://github.com/ocurrent/solver-service/pull/60.

The reason for the change is that sending two solve requests that differ only in [lower_bound] causes race conditions writing to the OCurrent job log. It is much more natural to instead represent a lower-bound build with an explicitly different platform.

Screenshot 2023-06-08 at 12 52 29