Skip to content

Fix supply tracer uncle accounting #31882

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix supply tracer uncle accounting #31882

wants to merge 1 commit into from

Conversation

sjb933
Copy link
Contributor

@sjb933 sjb933 commented May 23, 2025

Found a bug in the supply tracer, it's missing a fallthrough for the tracing.BalanceIncreaseRewardMineUncle case.

I added unit tests and fixed the issue 👍

@sjb933 sjb933 requested a review from s1na as a code owner May 23, 2025 00:39
@fjl fjl assigned s1na May 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@s1na s1na Awaiting requested review from s1na s1na is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@s1na s1na

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sjb933 @s1na