Merge #1305: docs: remove code-review comment

bc95fc4 docs: remove code-review comment (Jose Celano)

Pull request description:

  We decided not to change it.

  - The returned value is simpler.
  - We force the initial peer to change so there is no confusion about what was the final announced peer.

ACKs for top commit:
  josecelano:
    ACK bc95fc4

Tree-SHA512: 0973cc3ffed75befc8803c7e63271566f3a1c2046b631b42d67be8aa0cb674b06e63295b6d4c1a8f2b95a951ac8b9792564161388e90a2d518342b2faf9dd97b

Author: josecelano

packages/tracker-core/src/announce_handler.rs

@@ -162,10 +162,6 @@ impl AnnounceHandler {
         remote_client_ip: &IpAddr,
         peers_wanted: &PeersWanted,
     ) -> Result<AnnounceData, AnnounceError> {
-        // code-review: maybe instead of mutating the peer we could just return
-        // a tuple with the new peer and the announce data: (Peer, AnnounceData).
-        // It could even be a different struct: `StoredPeer` or `PublicPeer`.
-
         self.whitelist_authorization.authorize(info_hash).await?;
 
         tracing::debug!("Before: {peer:?}");