Repository navigation
heapq methods and asyncio.PriorityQueue missing appropriate comparison protocol bound #14418
Description
import heapq
class A:
...
x: list[A] = []
heapq.heappush(x, A())
heapq.heappush(x, A()) # will raise at runtime(similar cases constructable for other places where this is a problem)
The fix seems relatively trivial, so given that, I'm unsure if there's a reason this wasn't done already that would negatively impact existing use.
Surrounding context, this was noticed here,
Activity
srittau commentedon Jul 15, 2025
The relevant type var has been introduced ten years ago (3c4ace9), long before protocols were a thing, and never been revisited since then. PR welcome!
feat!: use stricter bounds for priority queues
feat!: use stricter bounds for priority queues