Skip to content

Conversation

@veeso
Copy link

@veeso veeso commented Dec 4, 2024

@veeso veeso marked this pull request as ready for review December 5, 2024 09:23

Choose a reason for hiding this comment

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

Maybe add at least one unit test that would show that all the calculated hashes have expected values?

Copy link

@ufoscout ufoscout left a comment

Choose a reason for hiding this comment

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

There are conflicts to be fixed

@veeso veeso requested a review from ufoscout January 8, 2025 10:26
@veeso veeso requested a review from Maximkaaa January 10, 2025 14:42
Copy link

Choose a reason for hiding this comment

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

Let's remove fmt changes to keep reth files as close to original as possible

@ufoscout
Copy link

ufoscout commented Feb 3, 2025

Superseded by EPROD-1132

@ufoscout ufoscout closed this Feb 3, 2025
@veeso veeso deleted the EPROD-1086-block-validation-with-a-single-reth-node branch February 7, 2025 08:58
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.

6 participants