build: updated_skeleton_tree_impl

[aner-starkware]

This change is 

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 11 lines in your changes are missing coverage. Please review.

Project coverage is 23.94%. Comparing base (8ec4d7a) to head (2dcea94).

Files	Patch %	Lines
.../src/patricia_merkle_tree/updated_skeleton_tree.rs	0.00%	11 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
- Coverage   28.33%   23.94%   -4.39%     
==========================================
  Files           6        7       +1     
  Lines          60       71      +11     
  Branches       60       71      +11     
==========================================
  Hits           17       17              
- Misses         43       54      +11     

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

[dorimedini-starkware]

Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @amosStarkware and @TzahiTaub)

[dorimedini-starkware]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @amosStarkware and @TzahiTaub)

[TzahiTaub]

crates/committer/src/patricia_merkle_tree/updated_skeleton_tree.rs line 22 at r3 (raw file):

struct UpdatedSkeletonTreeImpl<L: LeafDataTrait, H: HashFunction, TH: TreeHashFunction<L, H>> {
    skeleton_tree: HashMap<NodeIndex, UpdatedSkeletonNode<L>>,
    hash_function: H,

Non-blocking as it's out of this scope, but I don't think we need the hash functions in the updated (and Current) trees. These seem independent of the hash function. We should give them as args when calling compute_filled_tree.