ocurrent / current-bench

Experimental benchmarking infrastructure using OCurrent pipelines
Apache License 2.0
33 stars 17 forks source link

Fix broken GitHub notifications for changed metrics #470

Closed punchagan closed 1 year ago

punchagan commented 1 year ago

The code to figure out whether or not we should send a notification was unnecessarily complex and buggy. This commit simplifies the code, and fixes the bug which caused notifications to be not sent for repositories which didn't have a dockerfile configured in the .conf file, but had the default dockerfile (bench.Dockerfile) present in the repository.