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

Alt: Feat/load state fork test #14007

Closed
wants to merge 11 commits into from
Closed

Conversation

blmalone
Copy link
Contributor

Referencing another pr from external contributor: #13960. Kept all commit history

@blmalone blmalone requested a review from a team as a code owner January 27, 2025 22:56
@blmalone blmalone requested a review from maurelian January 27, 2025 22:56
Copy link

codecov bot commented Jan 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 42.71%. Comparing base (969382a) to head (e4c5002).
Report is 29 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #14007      +/-   ##
===========================================
- Coverage    46.96%   42.71%   -4.26%     
===========================================
  Files          973      807     -166     
  Lines        80990    71723    -9267     
  Branches       755        0     -755     
===========================================
- Hits         38035    30633    -7402     
+ Misses       39991    38396    -1595     
+ Partials      2964     2694     -270     
Flag Coverage Δ
cannon-go-tests-32 ?
cannon-go-tests-64 ?
contracts-bedrock-tests ?

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

see 173 files with indirect coverage changes

@blmalone blmalone closed this Jan 28, 2025
@blmalone
Copy link
Contributor Author

Closed this pr in favor of #14021 which CI passes on. Unknown why it fails on this pr.

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