Skip to content

Update core-contracts.v8#2010

Merged
gastonponti merged 5 commits intomasterfrom
gastonponti/update-contracts-v8
Feb 15, 2023
Merged

Update core-contracts.v8#2010
gastonponti merged 5 commits intomasterfrom
gastonponti/update-contracts-v8

Conversation

@gastonponti
Copy link
Copy Markdown
Contributor

Description

Update the branch that we use to track the last deployed core contracts on Mainnet

Other changes

Fix mycelo to handle all the changes. Add GrandaMento contract that was added before but we missed

Tested

Create the genesis block and run the tests

Related issues

  • Fixes #[issue number here]

Backwards compatibility

Yes

@gastonponti gastonponti requested a review from a team as a code owner February 14, 2023 12:33
@gastonponti gastonponti requested review from hbandura and piersy and removed request for a team and piersy February 14, 2023 12:33
@gastonponti gastonponti force-pushed the gastonponti/update-contracts-v8 branch from 75f1c22 to a164240 Compare February 14, 2023 13:02
@piersy
Copy link
Copy Markdown
Contributor

piersy commented Feb 14, 2023

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

coverage: 47.2% of statements across all listed packages
coverage:  52.6% of statements in consensus/istanbul
coverage:  40.0% of statements in consensus/istanbul/announce
coverage:  54.6% 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:  56.6% 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.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
CommentID: db2670421d

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 14, 2023

Codecov Report

Base: 54.82% // Head: 54.81% // Decreases project coverage by -0.02% ⚠️

Coverage data is based on head (a164240) compared to base (e831c00).
Patch coverage: 59.12% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2010      +/-   ##
==========================================
- Coverage   54.82%   54.81%   -0.02%     
==========================================
  Files         688      695       +7     
  Lines       92542    93258     +716     
==========================================
+ Hits        50737    51119     +382     
- Misses      37993    38299     +306     
- Partials     3812     3840      +28     
Impacted Files Coverage Δ
cmd/geth/config.go 0.00% <0.00%> (ø)
cmd/geth/main.go 20.00% <ø> (ø)
cmd/utils/flags.go 2.77% <0.00%> (-0.02%) ⬇️
consensus/istanbul/config.go 0.00% <0.00%> (ø)
consensus/istanbul/core/preprepare.go 71.23% <0.00%> (+7.03%) ⬆️
contracts/gasprice_minimum/gasprice_minimum.go 45.61% <0.00%> (-14.86%) ⬇️
core/sys_context.go 78.57% <ø> (ø)
core/vm/operations_acl.go 4.44% <ø> (ø)
eth/api_backend.go 0.00% <0.00%> (ø)
eth/downloader/queue.go 80.83% <ø> (-0.09%) ⬇️
... and 66 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@karlb
Copy link
Copy Markdown
Contributor

karlb commented Feb 15, 2023

This solves the genesis errors I was getting before.

@gastonponti gastonponti merged commit 78983db into master Feb 15, 2023
@gastonponti gastonponti deleted the gastonponti/update-contracts-v8 branch February 15, 2023 15:00
@palango palango mentioned this pull request Mar 10, 2023
palango pushed a commit that referenced this pull request Mar 10, 2023
Description
Update the branch that we use to track the last deployed core contracts on Mainnet

Other changes
Fix mycelo to handle all the changes. Add GrandaMento contract that was added before but we missed
gastonponti added a commit that referenced this pull request Mar 13, 2023
Description
Update the branch that we use to track the last deployed core contracts on Mainnet

Other changes
Fix mycelo to handle all the changes. Add GrandaMento contract that was added before but we missed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants