Skip to content

Comments

go: Update op-geth dependency to include Jovian receipt changes#18046

Merged
sebastianst merged 1 commit intodevelopfrom
seb/jovian-receipts
Oct 28, 2025
Merged

go: Update op-geth dependency to include Jovian receipt changes#18046
sebastianst merged 1 commit intodevelopfrom
seb/jovian-receipts

Conversation

@sebastianst
Copy link
Member

@sebastianst sebastianst commented Oct 28, 2025

Description

Updates op-geth dependency to ethereum-optimism/op-geth#710

Stacked on top of #18048

Tests

Adds assertions to DA footprint tests that the receipt fields are properly set.

@codecov
Copy link

codecov bot commented Oct 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (2fdd970) to head (97ce6f1).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #18046       +/-   ##
============================================
- Coverage    76.22%        0   -76.23%     
============================================
  Files          175        0      -175     
  Lines        10502        0    -10502     
============================================
- Hits          8005        0     -8005     
+ Misses        2351        0     -2351     
+ Partials       146        0      -146     
Flag Coverage Δ
cannon-go-tests-64 ?
contracts-bedrock-tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 175 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 force-pushed the seb/jovian-receipts branch 2 times, most recently from fc47744 to a423a0b Compare October 28, 2025 14:37
Copy link
Contributor

@joshklop joshklop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: if you stack the PRs, github will diff them nicely. I learned this from slipper recently. Not a big deal here since the diffs are small.

@sebastianst
Copy link
Member Author

nit: if you stack the PRs, github will diff them nicely. I learned this from slipper recently. Not a big deal here since the diffs are small.

Yep I'm aware. I just didn't bother since the first was already merging.

@sebastianst sebastianst force-pushed the seb/jovian-receipts branch 2 times, most recently from 2b4e02f to 97d826b Compare October 28, 2025 16:59
@sebastianst sebastianst marked this pull request as ready for review October 28, 2025 17:28
@sebastianst sebastianst requested review from a team as code owners October 28, 2025 17:28
@sebastianst sebastianst enabled auto-merge October 28, 2025 17:28
@sebastianst sebastianst added this pull request to the merge queue Oct 28, 2025
Merged via the queue into develop with commit 53432e2 Oct 28, 2025
67 checks passed
@sebastianst sebastianst deleted the seb/jovian-receipts branch October 28, 2025 18:08
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.

2 participants