ocaml-bench / sandmark

A benchmark suite for the OCaml compiler
The Unlicense
82 stars 40 forks source link

Run check_url only in CI instead of every run #414

Closed punchagan closed 1 year ago

punchagan commented 1 year ago

If a developer adds new files or changes any thing in the ocaml-versions/*.json files, they can manually run the check_url make target.

shakthimaan commented 1 year ago

There are OCaml compiler developers who do clone Sandmark and run locally on their development/test machines, and thus having the check built-in is useful. Adding it to the CI is fine, but, we need to keep it implicit for the depend target.

shakthimaan commented 1 year ago

LGTM!