Apply suggestions from code review

Co-authored-by: Arya <aryasolhi@gmail.com>

zebra-chain/src/block/commitment.rs

@@ -123,7 +123,7 @@ impl Commitment {
                 }
             }
             // NetworkUpgrade::current() returns the latest network upgrade that's activated at the provided height, so
-            // on Regtest for heights above height 0, it returns NU6, and it's possible for the current network upgrade
+            // on Regtest for heights above height 0, it could return NU6, and it's possible for the current network upgrade
             // to be NU6 (or Canopy, or any network upgrade above Heartwood) at the Heartwood activation height.
             (Canopy | Nu5 | Nu6, activation_height)
                 if height == activation_height

zebra-chain/src/transaction/arbitrary.rs

@@ -778,8 +778,6 @@ impl Arbitrary for Transaction {
             NetworkUpgrade::Blossom | NetworkUpgrade::Heartwood | NetworkUpgrade::Canopy => {
                 Self::v4_strategy(ledger_state)
             }
-            // TODO: move `NetworkUpgrade::Nu6` to its own match arm if there's a new tx format for
-            // NU6 or remove this TODO if there isn't.
             NetworkUpgrade::Nu5 | NetworkUpgrade::Nu6 => prop_oneof![
                 Self::v4_strategy(ledger_state.clone()),
                 Self::v5_strategy(ledger_state)

zebra-consensus/src/transaction/tests/prop.rs

@@ -344,8 +344,6 @@ fn sanitize_transaction_version(
             BeforeOverwinter => 2,
             Overwinter => 3,
             Sapling | Blossom | Heartwood | Canopy => 4,
-            // TODO: move `Nu6` to its own match arm if there's a new tx format for NU6 or remove
-            // this TODO if there isn't.
             Nu5 | Nu6 => 5,
         }
     };