ref: remove incr_version / decr_version (#15)

these methods are unsafe with our usage of cache

Author: asottile-sentry

django-stubs/core/cache/backends/base.pyi

@@ -60,10 +60,6 @@ class BaseCache:
     async def adelete_many(self, keys: Iterable[Any], version: Any | None = None) -> None: ...
     def clear(self) -> None: ...
     async def aclear(self) -> None: ...
-    def incr_version(self, key: Any, delta: int = 1, version: Any | None = None) -> int: ...
-    async def aincr_version(self, key: Any, delta: int = 1, version: Any | None = None) -> int: ...
-    def decr_version(self, key: Any, delta: int = 1, version: Any | None = None) -> int: ...
-    async def adecr_version(self, key: Any, delta: int = 1, version: Any | None = None) -> int: ...
     def close(self, **kwargs: Any) -> None: ...
     async def aclose(self, **kwargs: Any) -> None: ...
 

scripts/stubtest/allowlist.txt

@@ -232,6 +232,16 @@ django.contrib.auth.views.UserModel
 django.contrib.sessions.backends.base.SessionBase.get
 django.contrib.sessions.backends.base.SessionBase.setdefault
 
+# sentry: we remove these methods because they are unsafe for our usage
+django.core.cache.BaseCache.adecr_version
+django.core.cache.BaseCache.aincr_version
+django.core.cache.BaseCache.decr_version
+django.core.cache.BaseCache.incr_version
+django.core.cache.backends.base.BaseCache.adecr_version
+django.core.cache.backends.base.BaseCache.aincr_version
+django.core.cache.backends.base.BaseCache.decr_version
+django.core.cache.backends.base.BaseCache.incr_version
+
 # Limited stubtest support for enums (https://github.com/python/mypy/issues/16806)
 django.db.migrations.operations.base.OperationCategory.__new__