refactor: remove tree height field from updated skeleton tree

[nimrod-starkware]

This change is 

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.47%. Comparing base (b12c761) to head (482b2f5).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #191      +/-   ##
==========================================
- Coverage   61.61%   61.47%   -0.15%     
==========================================
  Files          36       36              
  Lines        1618     1612       -6     
  Branches     1618     1612       -6     
==========================================
- Hits          997      991       -6     
  Misses        571      571              
  Partials       50       50              

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

[dorimedini-starkware]

Reviewed 4 of 4 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @nimrod-starkware)

---

crates/committer/src/patricia_merkle_tree/updated_skeleton_tree/tree.rs line 41 at r1 (raw file):

pub(crate) struct UpdatedSkeletonTreeImpl {
    pub(crate) skeleton_tree: UpdatedSkeletonNodeMap,
}

Suggestion:

// TODO(Dori, 1/7/2024): Make this a tuple struct.
pub(crate) struct UpdatedSkeletonTreeImpl {
    pub(crate) skeleton_tree: UpdatedSkeletonNodeMap,
}

[nimrod-starkware]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dorimedini-starkware)

---

crates/committer/src/patricia_merkle_tree/updated_skeleton_tree/tree.rs line 41 at r1 (raw file):

pub(crate) struct UpdatedSkeletonTreeImpl {
    pub(crate) skeleton_tree: UpdatedSkeletonNodeMap,
}

Done.
Should i also add a todo to struct OriginalSkeletonTreeImpl?
it also has only one field.

[dorimedini-starkware]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @nimrod-starkware)

---

crates/committer/src/patricia_merkle_tree/updated_skeleton_tree/tree.rs line 41 at r1 (raw file):

Previously, nimrod-starkware wrote…

Done.
Should i also add a todo to struct OriginalSkeletonTreeImpl?
it also has only one field.

yes please

[nimrod-starkware]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dorimedini-starkware)

---

crates/committer/src/patricia_merkle_tree/updated_skeleton_tree/tree.rs line 41 at r1 (raw file):

Previously, dorimedini-starkware wrote…

yes please

Done.

[dorimedini-starkware]

Reviewed 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @nimrod-starkware)