Merge pull request #105 from voyc-jean/fix-django-6-incompatibility

Add Django 6 compatibility and modernise codebase

Author: stephrdev

.github/workflows/test.yml

@@ -11,7 +11,7 @@ on:
 jobs:
   tests:
     name: "Python ${{ matrix.python-version }}"
-    runs-on: "ubuntu-20.04"
+    runs-on: "ubuntu-latest"
     services:
       redis:
         image: redis
@@ -24,17 +24,17 @@ jobs:
           - 6379:6379
     strategy:
       matrix:
-        python-version: ["3.6", "3.7", "3.8", "3.9", "3.10"]
+        python-version: ["3.10", "3.11", "3.12", "3.13", "3.14"]
     steps:
-      - uses: "actions/checkout@v2"
-      - uses: "actions/setup-python@v2"
+      - uses: "actions/checkout@v6"
+      - uses: "actions/setup-python@v6"
         with:
           python-version: "${{ matrix.python-version }}"
       - name: "Install dependencies"
         run: |
           set -xe
           python -m pip install --upgrade pip setuptools
-          python -m pip install --upgrade poetry tox tox-gh-actions
+          python -m pip install --upgrade poetry tox tox-gh-actions poetry-plugin-export
 
       - name: "Run tox targets for ${{ matrix.python-version }}"
         run: "python -m tox"

.gitignore

@@ -1,4 +1,3 @@
-.vagrant
 sandbox/db.sqlite3
 docs/_build
 dist/*

.python-version

@@ -1,3 +1,5 @@
-3.8.5
-3.7.8
-3.6.11
+3.10.19
+3.11.14
+3.12.12
+3.13.11
+3.14.2

.readthedocs.yml

@@ -2,7 +2,7 @@ build:
   image: latest
 
 python:
-  version: 3.8
+  version: 3.14
   pip_install: true
   extra_requirements:
     - docs

CHANGELOG.rst

@@ -2,6 +2,16 @@
 Changelog
 =========
 
+4.0.0
+~~~~~
+
+* BREAKING CHANGE: Drop support for Django < 5.2 and Python < 3.10.
+* BREAKING CHANGE: Drop support for Celery < 5.0.
+* BREAKING CHANGE: Drop support for django-rq < 3.0.
+* Add support for Django 5.2.
+* Add support for Django 6.0.
+* Modernise Python code.
+
 3.0.0
 ~~~~~
 

README.rst

@@ -36,7 +36,7 @@ Yup - `over on readthedocs.org`_.
 Supported versions
 ------------------
 
-Python 3.6+ is supported. Django 2.2+ is supported.
+Python 3.10+ is supported. Django 5.2+ is supported.
 
 
 Do you have tests?

Vagrantfile

@@ -1,8 +1,5 @@
-try:
-    import importlib.metadata as importlib_metadata
-except ModuleNotFoundError:
-    # This is required for Python versions < 3.8
-    import importlib_metadata
+import importlib.metadata as importlib_metadata
+
 
 try:
     __version__ = importlib_metadata.version('django-cacheback')

cacheback/__init__.py

@@ -6,7 +6,6 @@
 from django.conf import settings
 from django.core.cache import DEFAULT_CACHE_ALIAS, caches
 from django.db.models import Model as DjangoModel
-from django.utils.itercompat import is_iterable
 
 from .utils import enqueue_task, get_job_class
 
@@ -29,15 +28,15 @@ def to_bytestring(value):
     :returns: a bytestring
     """
     if isinstance(value, DjangoModel):
-        return ('%s:%s' % (value.__class__, hash(value))).encode('utf-8')
+        return (f'{value.__class__}:{hash(value)}').encode('utf-8')
     if isinstance(value, str):
         return value.encode('utf8')
     if isinstance(value, bytes):
         return value
     return bytes(str(value), 'utf8')
 
 
-class Job(object):
+class Job:
     """
     A cached read job.
 
@@ -59,7 +58,7 @@ class Job(object):
     #: refresh the cache.
     refresh_timeout = 60
 
-    #: Secifies which cache to use from your `CACHES` setting. It defaults to
+    #: Specifies which cache to use from your `CACHES` setting. It defaults to
     #: `default`.
     cache_alias = None
 
@@ -91,7 +90,7 @@ class Job(object):
 
     @property
     def class_path(self):
-        return '%s.%s' % (self.__module__, self.__class__.__name__)
+        return f'{self.__module__}.{self.__class__.__name__}'
 
     def __init__(self):
         self.cache_alias = self.cache_alias or getattr(
@@ -399,11 +398,11 @@ def key(self, *args, **kwargs):
             return self.class_path
         try:
             if args and not kwargs:
-                return "%s:%s" % (self.class_path, self.hash(args))
+                return f"{self.class_path}:{self.hash(args)}"
             # The line might break if your passed values are un-hashable.  If
             # it does, you need to override this method and implement your own
             # key algorithm.
-            return "%s:%s:%s:%s" % (
+            return "{}:{}:{}:{}".format(
                 self.class_path,
                 self.hash(args),
                 self.hash([k for k in sorted(kwargs)]),
@@ -422,7 +421,7 @@ def hash(self, value):
 
         This is for use in a cache key.
         """
-        if is_iterable(value):
+        if isinstance(value, collections.abc.Iterable):
             value = tuple(to_bytestring(v) for v in value)
         return hashlib.md5(b':'.join(value)).hexdigest()
 
@@ -440,7 +439,7 @@ def process_result(self, result, call, cache_status, sync_fetch=None):
         :param result: The result to be returned
         :param call: A named tuple with properties 'args' and 'kwargs that
                      holds the call args and kwargs
-        :param cache_status: A status integrer, accessible as class constants
+        :param cache_status: A status integer, accessible as class constants
                              self.MISS, self.HIT, self.STALE
         :param sync_fetch: A boolean indicating whether a synchronous fetch was
                            performed. A value of None indicates that no fetch

cacheback/base.py

@@ -1,7 +1,4 @@
-try:
-    import importlib
-except ImportError:
-    import django.utils.importlib as importlib
+import importlib
 
 from .base import Job
 
@@ -19,7 +16,7 @@ def __init__(
         task_options=None,
         set_data_kwarg=None,
     ):
-        super(FunctionJob, self).__init__()
+        super().__init__()
         if lifetime is not None:
             self.lifetime = int(lifetime)
         if fetch_on_miss is not None:
@@ -43,7 +40,7 @@ def get_init_kwargs(self):
     def prepare_args(self, fn, *args):
         # Convert function into "module:name" form so that is can be pickled and
         # then re-imported.
-        return ("%s:%s" % (fn.__module__, fn.__name__),) + args
+        return (f"{fn.__module__}:{fn.__name__}",) + args
 
     def fetch(self, fn_string, *args, **kwargs):
         # Import function from string representation
@@ -67,7 +64,7 @@ def __init__(
         """
         :model: The model class to use
         """
-        super(QuerySetJob, self).__init__()
+        super().__init__()
         self.model = model
         if lifetime is not None:
             self.lifetime = lifetime
@@ -82,7 +79,7 @@ def get_init_kwargs(self):
         return {'model': self.model, 'lifetime': self.lifetime, 'cache_alias': self.cache_alias}
 
     def key(self, *args, **kwargs):
-        return "%s-%s" % (self.model.__name__, super(QuerySetJob, self).key(*args, **kwargs))
+        return f"{self.model.__name__}-{super().key(*args, **kwargs)}"
 
 
 class QuerySetGetJob(QuerySetJob):