Skip to content

Commit

Permalink
ci: Fix incorrect coverage reported (#715)
Browse files Browse the repository at this point in the history
We computed everything correctly, but printed the old value twice >.<
  • Loading branch information
aborgna-q authored Nov 23, 2023
1 parent b7a5176 commit c52ed51
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/notify-coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ jobs:
if: steps.get_coverage.outputs.should_notify == 'true'
run: |
prev=`cat coverage-prev.txt`
current=`cat coverage-prev.txt`
current=`cat coverage.txt`
change=`printf "%.2f%% --> %.2f%%" $prev $current`
codecov="https://codecov.io/gh/${{ github.repository }}?search=&trend=7%20days"
if (( $(echo "$prev < $current + 0.04" | bc -l) ))
Expand Down

0 comments on commit c52ed51

Please sign in to comment.