Skip to content

Conversation

sebastianst
Copy link
Member

@sebastianst sebastianst commented May 9, 2025

Description

Updates op-geth dependency to include upstream geth v1.15.10 changes from ethereum-optimism/op-geth#593.

Stacked on top of #15587

Additional context

Metadata

@sebastianst sebastianst self-assigned this May 9, 2025
Copy link

codecov bot commented May 9, 2025

Codecov Report

Attention: Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.

Project coverage is 44.62%. Comparing base (2bdaa8c) to head (a3ec173).

Files with missing lines Patch % Lines
op-program/client/l2/engineapi/block_processor.go 0.00% 6 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (2bdaa8c) and HEAD (a3ec173). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (2bdaa8c) HEAD (a3ec173)
contracts-bedrock-tests 1 0
Additional details and impacted files
@@                  Coverage Diff                  @@
##           seb/geth-v1.15.7   #15810       +/-   ##
=====================================================
- Coverage             81.21%   44.62%   -36.60%     
=====================================================
  Files                   161     1230     +1069     
  Lines                  8756   103556    +94800     
=====================================================
+ Hits                   7111    46208    +39097     
- Misses                 1512    53744    +52232     
- Partials                133     3604     +3471     
Flag Coverage Δ
cannon-go-tests-64 65.37% <ø> (ø)
contracts-bedrock-tests ?

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

Files with missing lines Coverage Δ
op-program/client/l2/engineapi/block_processor.go 51.53% <0.00%> (ø)

... and 1280 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sebastianst sebastianst requested a review from geoknee May 13, 2025 10:55
@sebastianst sebastianst marked this pull request as ready for review May 13, 2025 11:11
@sebastianst sebastianst requested review from a team as code owners May 13, 2025 11:11
@sebastianst sebastianst requested review from pauldowman and removed request for a team May 13, 2025 11:11
@sebastianst sebastianst added the M-do-not-merge Meta: Do not merge label May 13, 2025
@sebastianst
Copy link
Member Author

#15829

@sebastianst sebastianst deleted the geth/v1.15.10 branch May 21, 2025 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
M-do-not-merge Meta: Do not merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants