fix: address review — remove **_kwargs, set docstring on wrapper, drop deprecation from dev deps

- Remove **_kwargs parameter from docstring_deprecated() since all
  call sites use only 'details' and 'deprecated_in' keyword args
- Move docstring modification to wrapper instead of original func
  to preserve the original function's __doc__
- Remove deprecation==2.1.0 from requirements-devel.txt to fully
  eliminate the unmaintained dependency

Signed-off-by: NIK-TIGER-BILL <nik.tiger.bill@github.com>

Author: NIK-TIGER-BILL

requirements-devel.txt

@@ -5,7 +5,6 @@ grpcio==1.75.1
 grpcio-tools==1.75.1
 grpcio-health-checking==1.75.1
 pydantic==2.12.0
-deprecation==2.1.0
 
 build
 twine

weaviate/util.py

@@ -34,7 +34,7 @@
 BYTES_PER_CHUNK = 65535  # The number of bytes to read per chunk when encoding files ~ 64kb
 
 
-def docstring_deprecated(details: str = "", deprecated_in: str = "", **_kwargs: object) -> Callable:
+def docstring_deprecated(details: str = "", deprecated_in: str = "") -> Callable:
     """Stdlib replacement for ``deprecation.deprecated``.
 
     The ``deprecation`` package has not been maintained since 2019 and is
@@ -46,10 +46,6 @@ def docstring_deprecated(details: str = "", deprecated_in: str = "", **_kwargs:
     """
 
     def decorator(func: Callable) -> Callable:
-        docstring = func.__doc__ or ""
-        note = f".. deprecated:: {deprecated_in}\n   {details}\n\n" if deprecated_in else ""
-        func.__doc__ = note + docstring
-
         @functools.wraps(func)
         def wrapper(*args: object, **kwargs: object) -> object:
             _warnings.warn(
@@ -59,6 +55,10 @@ def wrapper(*args: object, **kwargs: object) -> object:
             )
             return func(*args, **kwargs)
 
+        docstring = func.__doc__ or ""
+        note = f".. deprecated:: {deprecated_in}\n   {details}\n\n" if deprecated_in else ""
+        wrapper.__doc__ = note + docstring
+
         return wrapper  # type: ignore[return-value]
 
     return decorator