Don't use hash to avoid attrs warning (#3054)

Author: CoolCat467

newsfragments/3053.bugfix.rst

@@ -0,0 +1 @@
+Switched ``attrs`` usage off of ``hash``, which is now deprecated.

pyproject.toml

@@ -40,7 +40,7 @@ dependencies = [
     # attrs 19.2.0 adds `eq` option to decorators
     # attrs 20.1.0 adds @frozen
     # attrs 21.1.0 adds a dataclass transform for type-checkers
-    # attrs 21.3.0 adds `import addrs`
+    # attrs 21.3.0 adds `import attrs`
     "attrs >= 23.2.0",
     "sortedcontainers",
     "idna",

src/trio/_core/_entry_queue.py

@@ -146,7 +146,7 @@ def run_sync_soon(
 
 
 @final
-@attrs.define(eq=False, hash=False)
+@attrs.define(eq=False)
 class TrioToken(metaclass=NoPublicConstructor):
     """An opaque object representing a single call to :func:`trio.run`.
 

src/trio/_core/_io_epoll.py

@@ -198,7 +198,7 @@ class _EpollStatistics:
 # wanted to about how epoll works.
 
 
-@attrs.define(eq=False, hash=False)
+@attrs.define(eq=False)
 class EpollIOManager:
     # Using lambda here because otherwise crash on import with gevent monkey patching
     # See https://github.com/python-trio/trio/issues/2848

src/trio/_core/_local.py

@@ -16,7 +16,7 @@ class _NoValue: ...
 
 
 @final
-@attrs.define(eq=False, hash=False)
+@attrs.define(eq=False)
 class RunVarToken(Generic[T], metaclass=NoPublicConstructor):
     _var: RunVar[T]
     previous_value: T | type[_NoValue] = _NoValue
@@ -28,7 +28,7 @@ def _empty(cls, var: RunVar[T]) -> RunVarToken[T]:
 
 
 @final
-@attrs.define(eq=False, hash=False, repr=False)
+@attrs.define(eq=False, repr=False)
 class RunVar(Generic[T]):
     """The run-local variant of a context variable.
 

src/trio/_core/_parking_lot.py

@@ -101,7 +101,7 @@ class ParkingLotStatistics:
 
 
 @final
-@attrs.define(eq=False, hash=False)
+@attrs.define(eq=False)
 class ParkingLot:
     """A fair wait queue with cancellation and requeueing.
 

src/trio/_core/_run.py

@@ -845,7 +845,7 @@ def started(self, value: StatusT_contra | None = None) -> None:
 
 # This code needs to be read alongside the code from Nursery.start to make
 # sense.
-@attrs.define(eq=False, hash=False, repr=False, slots=False)
+@attrs.define(eq=False, repr=False, slots=False)
 class _TaskStatus(TaskStatus[StatusT]):
     _old_nursery: Nursery
     _new_nursery: Nursery
@@ -1289,7 +1289,7 @@ def __del__(self) -> None:
 
 
 @final
-@attrs.define(eq=False, hash=False, repr=False)
+@attrs.define(eq=False, repr=False)
 class Task(metaclass=NoPublicConstructor):
     _parent_nursery: Nursery | None
     coro: Coroutine[Any, Outcome[object], Any]
@@ -1532,7 +1532,7 @@ class RunStatistics:
 # worker thread.
 
 
-@attrs.define(eq=False, hash=False)
+@attrs.define(eq=False)
 class GuestState:
     runner: Runner
     run_sync_soon_threadsafe: Callable[[Callable[[], object]], object]
@@ -1582,7 +1582,7 @@ def in_main_thread() -> None:
             start_thread_soon(get_events, deliver)
 
 
-@attrs.define(eq=False, hash=False)
+@attrs.define(eq=False)
 class Runner:
     clock: Clock
     instruments: Instruments

src/trio/_core/_tests/test_instrumentation.py

@@ -12,7 +12,7 @@
     from ...lowlevel import Task
 
 
-@attrs.define(eq=False, hash=False, slots=False)
+@attrs.define(eq=False, slots=False)
 class TaskRecorder(_abc.Instrument):
     record: list[tuple[str, Task | None]] = attrs.Factory(list)
 

src/trio/_highlevel_generic.py

@@ -53,7 +53,7 @@ def _is_halfclosable(stream: SendStream) -> TypeGuard[HalfCloseableStream]:
 
 
 @final
-@attrs.define(eq=False, hash=False, slots=False)
+@attrs.define(eq=False, slots=False)
 class StapledStream(
     HalfCloseableStream,
     Generic[SendStreamT, ReceiveStreamT],

src/trio/_sync.py

@@ -33,7 +33,7 @@ class EventStatistics:
 
 
 @final
-@attrs.define(repr=False, eq=False, hash=False)
+@attrs.define(repr=False, eq=False)
 class Event:
     """A waitable boolean value useful for inter-task synchronization,
     inspired by :class:`threading.Event`.
@@ -538,7 +538,7 @@ class LockStatistics:
     tasks_waiting: int
 
 
-@attrs.define(eq=False, hash=False, repr=False, slots=False)
+@attrs.define(eq=False, repr=False, slots=False)
 class _LockImpl(AsyncContextManagerMixin):
     _lot: ParkingLot = attrs.field(factory=ParkingLot, init=False)
     _owner: Task | None = attrs.field(default=None, init=False)