Skip to content

internal/ethapi, miner: fix GetBlockReceipts for pending#32461

Merged
lightclient merged 4 commits intoethereum:masterfrom
rjl493456442:support-pending-block-receipts
Aug 20, 2025
Merged

internal/ethapi, miner: fix GetBlockReceipts for pending#32461
lightclient merged 4 commits intoethereum:masterfrom
rjl493456442:support-pending-block-receipts

Conversation

@rjl493456442
Copy link
Copy Markdown
Member

The alternative fix for #32456

This pull request is opened due to the lacking permission to directly edit the
original PR from berachain. #32457

@calbera
Copy link
Copy Markdown

calbera commented Aug 18, 2025

@rjl493456442 thank you for pushing this up. I like this refactor since pending receipts will never come from disk!

Closes #32456

lightclient
lightclient previously approved these changes Aug 18, 2025
Copy link
Copy Markdown
Member

@lightclient lightclient left a comment

Choose a reason for hiding this comment

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

Generally SGTM

chain *core.BlockChain
accman *accounts.Manager
acc accounts.Account
generator func(i int, b *core.BlockGen)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why is this needed?

@rjl493456442
Copy link
Copy Markdown
Member Author

@lightclient I have removed the unused generator, please approve it again!

@rjl493456442 rjl493456442 added this to the 1.16.3 milestone Aug 20, 2025
@lightclient lightclient merged commit 9ce40d1 into ethereum:master Aug 20, 2025
4 of 5 checks passed
gballet pushed a commit to gballet/go-ethereum that referenced this pull request Sep 11, 2025
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