search

ocaml-bench / sandmark

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

Skip running build if benchmarks exist for a given commit #383

Closed punchagan closed 2 years ago

punchagan commented 2 years ago

Closes ocaml-bench/sandmark-nightly-config#6

This check may be too aggressive, i.e., manually running the run_all_custom.sh script wouldn't work if a nightly build for that commit already exists, and similarly manual test builds would cause the nightly to skip builds, etc. @shakthimaan let me know if this seems too aggressive to you and if we should skip builds for old commits differently.

punchagan commented 2 years ago

Updated this PR to check if the "summary.bench" files for a specific commit have been committed on the main branch of sandmark-nightly to decide if we should skip running builds for it.

shakthimaan commented 2 years ago

LGTM!