Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

port pr6271 to migration mainnet branch #6409

Closed
wants to merge 2 commits into from
Closed

Conversation

j1010001
Copy link
Member

porting #6271 to feature/migration-mainnet branch - those changes are needed for bootstrapping mainnet state clone on migration mainnet.

@j1010001 j1010001 closed this Aug 27, 2024
@j1010001 j1010001 deleted the jan/port-pr6271 branch August 27, 2024 21:40
@codecov-commenter
Copy link

codecov-commenter commented Aug 27, 2024

Codecov Report

Attention: Patch coverage is 71.90742% with 352 lines in your changes missing coverage. Please review.

Project coverage is 43.52%. Comparing base (2a1e71e) to head (a398a52).
Report is 44 commits behind head on master.

Files Patch % Lines
fvm/environment/program_recovery.go 4.28% 134 Missing ⚠️
fvm/evm/impl/impl.go 76.82% 69 Missing and 61 partials ⚠️
fvm/evm/impl/abi.go 83.26% 50 Missing and 33 partials ⚠️
fvm/environment/mock/environment.go 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6409      +/-   ##
==========================================
+ Coverage   41.52%   43.52%   +1.99%     
==========================================
  Files        2013     1804     -209     
  Lines      143520   124264   -19256     
==========================================
- Hits        59596    54085    -5511     
+ Misses      77748    64574   -13174     
+ Partials     6176     5605     -571     
Flag Coverage Δ
unittests 43.52% <71.90%> (+1.99%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

3 participants