Skip to content

Commit d2fdf70

Browse files
authored
Merge pull request #819 from lla-dane/remove-todos
Remove completed TODO task comments in Peerstore
2 parents fc9b289 + f4247fa commit d2fdf70

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

libp2p/peer/peerstore.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,6 @@
3939
PERMANENT_ADDR_TTL = 0
4040

4141

42-
# TODO: Set up an async task for periodic peer-store cleanup
43-
# for expired addresses and records.
4442
class PeerRecordState:
4543
envelope: Envelope
4644
seq: int
@@ -217,7 +215,6 @@ def clear_metadata(self, peer_id: ID) -> None:
217215

218216
# -----CERT-ADDR-BOOK-----
219217

220-
# TODO: Make proper use of this function
221218
def maybe_delete_peer_record(self, peer_id: ID) -> None:
222219
"""
223220
Delete the signed peer record for a peer if it has no know

newsfragments/819.internal.rst

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
Remove the already completed TODO tasks in Peerstore:
2+
TODO: Set up an async task for periodic peer-store cleanup for expired addresses and records.
3+
TODO: Make proper use of this function

0 commit comments

Comments
 (0)