calliope-project / calliope

A multi-scale energy systems modelling framework
https://www.callio.pe
Apache License 2.0
276 stars 89 forks source link

Fix: Update commit-ci codecov step #629

Closed brynpickering closed 2 days ago

brynpickering commented 3 days ago

Fixes codecov not uploading successfully on main and its failure being silent. This is why we have failing coverage reports in all our PRs.

Summary of changes in this pull request

Reviewer checklist

codecov[bot] commented 3 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.35%. Comparing base (872978d) to head (181318c). Report is 21 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #629 +/- ## ========================================== - Coverage 95.85% 95.35% -0.51% ========================================== Files 24 24 Lines 3619 3656 +37 Branches 788 734 -54 ========================================== + Hits 3469 3486 +17 + Misses 86 82 -4 - Partials 64 88 +24 ``` [see 3 files with indirect coverage changes](https://app.codecov.io/gh/calliope-project/calliope/pull/629/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=calliope-project)
brynpickering commented 2 days ago

Exactly. You can see it failing in this CI job due to lack of an API token, but doing so with a warning that gets passed over silently.