Skip to content

Conversation

xin-hedera
Copy link
Contributor

Description:

This PR updates the types of items included for block root hash calculation

  • Include BlockHeader and RoundHeader for block root hash calcuation

Related issue(s):

Fixes #10075

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

…ockheader-roundheader-to-block-item-merkle-tree-for-block-root-hash-calculation

Signed-off-by: Xin Li <[email protected]>
@xin-hedera xin-hedera added this to the 0.124.0 milestone Feb 7, 2025
@xin-hedera xin-hedera self-assigned this Feb 7, 2025
@xin-hedera xin-hedera requested a review from a team as a code owner February 7, 2025 20:15
@xin-hedera xin-hedera added enhancement Type: New feature importer Area: Importer labels Feb 7, 2025
Copy link

codecov bot commented Feb 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.21%. Comparing base (e67266d) to head (ad143c2).
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main   #10322   +/-   ##
=========================================
  Coverage     92.21%   92.21%           
  Complexity     8024     8024           
=========================================
  Files           982      982           
  Lines         33507    33508    +1     
  Branches       4226     4226           
=========================================
+ Hits          30897    30900    +3     
+ Misses         1609     1607    -2     
  Partials       1001     1001           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

sonarqubecloud bot commented Feb 7, 2025

Copy link
Contributor

@steven-sheehy steven-sheehy left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@edwin-greene edwin-greene left a comment

Choose a reason for hiding this comment

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

Looks good

Copy link
Contributor

@jnels124 jnels124 left a comment

Choose a reason for hiding this comment

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

lgtm

@steven-sheehy steven-sheehy merged commit ed7574e into main Feb 10, 2025
35 of 36 checks passed
@steven-sheehy steven-sheehy deleted the 10075-hip-1056-add-blockheader-roundheader-to-block-item-merkle-tree-for-block-root-hash-calculation branch February 10, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type: New feature importer Area: Importer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HIP-1056 Add BlockHeader / RoundHeader to block item merkle tree for block root hash calculation
4 participants