Delete outdated TODOs refering to closed issues

zebra-chain/src/work/difficulty/tests/vectors.rs

@@ -457,7 +457,7 @@ fn check_testnet_minimum_difficulty_block(height: block::Height) -> Result<(), R
             .signed_duration_since(previous_block.header.time);
 
         // zcashd requires a gap that's strictly greater than 6 times the target
-        // threshold, but ZIP-205 and ZIP-208 are ambiguous. See bug #1276.
+        // threshold.
         match NetworkUpgrade::minimum_difficulty_spacing_for_height(Network::Testnet, height) {
             None => Err(eyre!("the minimum difficulty rule is not active"))?,
             Some(spacing) if (time_gap <= spacing) => Err(eyre!(

zebra-consensus/src/transaction.rs

@@ -740,10 +740,6 @@ where
             orchard_shielded_data,
             &shielded_sighash,
         )?))
-
-        // TODO:
-        // - verify orchard shielded pool (ZIP-224) (#2105)
-        // - shielded input and output limits? (#2379)
     }
 
     /// Verifies if a V5 `transaction` is supported by `network_upgrade`.

zebra-network/src/peer_set/limit.rs

@@ -178,8 +178,6 @@ impl Drop for ConnectionTracker {
 
         // We ignore disconnected errors, because the receiver can be dropped
         // before some connections are dropped.
-        //
-        // TODO: This channel will be bounded by the connection limit (#1850, #1851, #2902).
         let _ = self.close_notification_tx.send(ConnectionClosed);
     }
 }

zebra-network/src/peer_set/set.rs

@@ -504,7 +504,6 @@ where
     /// Checks if the minimum peer version has changed, and disconnects from outdated peers.
     fn disconnect_from_outdated_peers(&mut self) {
         if let Some(minimum_version) = self.minimum_peer_version.changed() {
-            // TODO: Remove when the code base migrates to Rust 2021 edition (#2709).
             let preselected_p2c_peer = &mut self.preselected_p2c_peer;
 
             self.ready_services.retain(|address, peer| {

zebra-network/src/policies.rs

@@ -34,9 +34,6 @@ impl<Req: Clone + std::fmt::Debug, Res, E: std::fmt::Debug> Policy<Req, Res, E>
                 Some(
                     // Let other tasks run, so we're more likely to choose a different peer,
                     // and so that any notfound inv entries win the race to the PeerSet.
-                    //
-                    // TODO: move syncer retries into the PeerSet,
-                    //       so we always choose different peers (#3235)
                     Box::pin(tokio::task::yield_now().map(move |()| retry_outcome)),
                 )
             } else {

zebra-state/src/service/finalized_state/tests/prop.rs

@@ -35,7 +35,6 @@ fn blocks_with_v5_transactions() -> Result<()> {
                 );
                 prop_assert_eq!(Some(height), state.finalized_tip_height());
                 prop_assert_eq!(hash.unwrap(), block.hash);
-                // TODO: check that the nullifiers were correctly inserted (#2230)
                 height = Height(height.0 + 1);
             }
     });