ocaml-semver / ocaml-api-watch

Libraries and tools to keep watch on you OCaml lib's API changes
ISC License
21 stars 15 forks source link

Update to remove install error #15

Closed rmeis06 closed 8 months ago

rmeis06 commented 8 months ago

Hello! I'm an outreachy applicant.

While following the instructions in CONTRIBUTING.md I got the following error:

Switch invariant: ["ocaml-base-compiler" {= "4.12.1"} | "ocaml-system" {= "4.12.1"}]
[ERROR] Could not determine which packages to install for this switch:
  * No agreement on the version of ocaml:
    - (invariant) → ocaml-base-compiler = 4.12.1 → ocaml = 4.12.1
    - api-watch → ocaml >= 4.14.0
    You can temporarily relax the switch invariant with `--update-invariant'

Which I resolved by making the change in the PR to my install. Would this change be correct for the install instructions to be working for others as well?

shonfeder commented 8 months ago

Awesome work! Thanks again for the fix :)