search

beacon-biosignals / K8sClusterManagers.jl

A Julia cluster manager for Kubernetes
Other
31 stars 5 forks source link

Change Codecov comment behaviour #45

Closed omus closed 3 years ago

omus commented 3 years ago

It appears that sometimes codecov doesn't update the comment to reflect the current coverage status. This behaviour should make it more obvious if the comment has been updated as the old comment will be deleted and a new one posted.

omus commented 3 years ago

Ugh, this seems to have just stopped Codecov...

omus commented 3 years ago

I discovered that the Codecov webhook was encountering 403 errors so I deleted the webhook and this repo is now setup with the Codecov app: https://github.com/apps/codecov

omus commented 3 years ago

It appears to me that Codecov is ignoring the default require_ci_to_pass: true and is posting each time any CI run finishes

codecov[bot] commented 3 years ago

Codecov Report

Merging #45 (715c4e4) into main (d882905) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #45   +/-   ##
=======================================
  Coverage   52.81%   52.81%           
=======================================
  Files           4        4           
  Lines         142      142           
=======================================
  Hits           75       75           
  Misses         67       67

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update d882905...715c4e4. Read the comment docs.