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

Update to contracts v10 #2139

Closed karlb closed 1 year ago

karlb commented 1 year ago

Since we want to wrap up the Gingerbread fork soo, we should start to depend on the branches in the monorepo which will eventually be used for the fork:

I also added the monorepo commits we depend on which have not been merged into the v10 contracts branch yet. See https://github.com/celo-org/celo-monorepo/tree/karlb/core-contracts.v10-ultragreen

So main part of this PR is actually in the changed monorepo commit and the changes to this repo are quite simple.

github-actions[bot] commented 1 year ago
5841 passed, 45 skipped
codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 66.55% and project coverage change: +1.02 :tada:

Comparison is base (71bdbcf) 54.30% compared to head (de3f84b) 55.32%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2139 +/- ## ========================================== + Coverage 54.30% 55.32% +1.02% ========================================== Files 692 675 -17 Lines 115642 113730 -1912 ========================================== + Hits 62795 62923 +128 + Misses 49014 46966 -2048 - Partials 3833 3841 +8 ``` | [Impacted Files](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org) | Coverage Δ | | |---|---|---| | [cmd/devp2p/internal/ethtest/transaction.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-Y21kL2RldnAycC9pbnRlcm5hbC9ldGh0ZXN0L3RyYW5zYWN0aW9uLmdv) | `0.00% <0.00%> (ø)` | | | [cmd/evm/runner.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-Y21kL2V2bS9ydW5uZXIuZ28=) | `0.00% <0.00%> (ø)` | | | [cmd/evm/staterunner.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-Y21kL2V2bS9zdGF0ZXJ1bm5lci5nbw==) | `0.00% <0.00%> (ø)` | | | [cmd/geth/chaincmd.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-Y21kL2dldGgvY2hhaW5jbWQuZ28=) | `0.00% <0.00%> (ø)` | | | [cmd/geth/config.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-Y21kL2dldGgvY29uZmlnLmdv) | `0.00% <0.00%> (ø)` | | | [cmd/geth/main.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-Y21kL2dldGgvbWFpbi5nbw==) | `21.39% <ø> (+0.91%)` | :arrow_up: | | [cmd/geth/usage.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-Y21kL2dldGgvdXNhZ2UuZ28=) | `10.90% <ø> (ø)` | | | [cmd/utils/flags.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-Y21kL3V0aWxzL2ZsYWdzLmdv) | `2.56% <0.00%> (ø)` | | | [consensus/istanbul/utils.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-Y29uc2Vuc3VzL2lzdGFuYnVsL3V0aWxzLmdv) | `47.10% <ø> (+1.13%)` | :arrow_up: | | [contracts/currency/currency.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-Y29udHJhY3RzL2N1cnJlbmN5L2N1cnJlbmN5Lmdv) | `55.00% <ø> (ø)` | | | ... and [62 more](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2139?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org) | | ... and [28 files with indirect coverage changes](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2139/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

github-actions[bot] commented 1 year ago

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

coverage: 49.9% of statements across all listed packages
coverage:  59.8% of statements in consensus/istanbul
coverage:  43.1% of statements in consensus/istanbul/announce
coverage:  55.7% 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:  63.0% of statements in consensus/istanbul/core
coverage:  50.0% of statements in consensus/istanbul/db
coverage:   0.0% of statements in consensus/istanbul/proxy
coverage:  64.4% 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