celo-org / celo-blockchain

Official repository for the golang Celo Blockchain
https://celo.org
GNU Lesser General Public License v3.0
560 stars 198 forks source link

CI: Fix forkdiff build #2212

Closed palango closed 10 months ago

palango commented 10 months ago

Description

In https://github.com/celo-org/celo-blockchain/actions/runs/7260906287/job/19781026224 the forkdiff job fails because of an old go version. This upgrade to version to fix the build.

github-actions[bot] commented 10 months ago

Coverage from tests in ./e2e_test/... for ./consensus/istanbul/... at commit 44c749030bb53112c94a99c9e5ce347517a67c52

coverage: 46.9% of statements across all listed packages
coverage:  57.2% of statements in consensus/istanbul
coverage:  23.7% of statements in consensus/istanbul/announce
coverage:  54.3% of statements in consensus/istanbul/backend
coverage:   0.0% of statements in consensus/istanbul/backend/backendtest
coverage:  24.3% of statements in consensus/istanbul/backend/internal/replica
coverage:  65.0% of statements in consensus/istanbul/core
coverage:  45.0% of statements in consensus/istanbul/db
coverage:   0.0% of statements in consensus/istanbul/proxy
coverage:  64.2% of statements in consensus/istanbul/uptime
coverage:  51.8% of statements in consensus/istanbul/validator
coverage:  79.2% of statements in consensus/istanbul/validator/random
github-actions[bot] commented 10 months ago

5878 passed, 2 failed, 45 skipped

Test failures:
  TestTransferERC20: e2e_test

Checking getExchangeSpenders. spenders = [0x000000000000000000000000000000000000d028]
Checking medianRate. numerator = 1000000000000000000000000  denominator = 1000000000000000000000000 
Checking gas price minimum. cusdValue = 100000000
    e2e_transfer_test.go:472: 
  TestStartStopValidators: e2e_test
Checking getExchangeSpenders. spenders = [0x000000000000000000000000000000000000d028]
Checking medianRate. numerator = 1000000000000000000000000  denominator = 1000000000000000000000000 
Checking gas price minimum. cusdValue = 100000000
    e2e_test.go:286: 
This test report was produced by the test-summary action.  Made with ❤️ in Cambridge.
codecov[bot] commented 10 months ago

Codecov Report

Attention: 48 lines in your changes are missing coverage. Please review.

Comparison is base (6731538) 55.09% compared to head (222e8ce) 55.10%. Report is 2 commits behind head on master.

Files Patch % Lines
cmd/utils/flags.go 0.00% 25 Missing :warning:
miner/block.go 44.82% 14 Missing and 2 partials :warning:
core/sys_context.go 0.00% 7 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2212 +/- ## ======================================= Coverage 55.09% 55.10% ======================================= Files 682 683 +1 Lines 114449 114533 +84 ======================================= + Hits 63055 63109 +54 - Misses 47501 47530 +29 - Partials 3893 3894 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.