chore: remove deprecated pre_save

.github/workflows/migration-fixer.yml

@@ -17,7 +17,9 @@ jobs:
       - name: Set up Python
         uses: actions/setup-python@v4
         with:
-          python-version: '3.6.x'
+          python-version: '3.7.x'
+          cache: 'pip'
+          cache-dependency-path: '**/requirements.txt'
 
       - name: Install dependencies
         run: |

.github/workflows/test.yml

@@ -23,9 +23,11 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        platform: [ubuntu-latest, macos-latest, windows-latest]
+        platform: [ubuntu-20.04, ubuntu-latest, macos-latest, windows-latest]
         python-version: [3.6, 3.7, 3.8, 3.9, '3.10', 3.11]
         exclude:
+          - platform: ubuntu-latest
+            python-version: 3.6
           - platform: macos-latest
             python-version: 3.11
           - platform: windows-latest

model_clone/mixin.py

@@ -1,5 +1,4 @@
 import itertools
-import warnings
 from itertools import repeat
 
 from conditional import conditional
@@ -196,23 +195,6 @@ def bulk_clone_multi(cls, objs, attrs=None, batch_size=None):
         # TODO: Support bulk clones split by the batch_size
         pass
 
-    def pre_save_duplicate(self, instance):  # pylint: disable=R0201
-        """
-        This method has been deprecated and would be removed in the 5.0.0 release.
-        Please use `pre_clone_save` signal instead.
-
-        See: https://github.com/tj-django/django-clone#signals for guidance.
-        """
-        # TODO: Remove prior to v5.0.0 release
-        warnings.warn(
-            "The usage of `pre_save_duplicate` has been deprecated "
-            "and would be removed in the 5.0.0 release "
-            "Please use `pre_clone_save` signal instead.",
-            DeprecationWarning,
-            stacklevel=2,
-        )
-        return instance
-
     @transaction.atomic
     def make_clone(self, attrs=None, sub_clone=False, using=None, parent=None):
         """Creates a clone of the django model instance.
@@ -245,7 +227,6 @@ def make_clone(self, attrs=None, sub_clone=False, using=None, parent=None):
         for name, value in attrs.items():
             setattr(duplicate, name, value)
 
-        duplicate = self.pre_save_duplicate(duplicate)
         duplicate = self.__duplicate_m2o_fields(duplicate, using=using)
 
         pre_clone_save.send(sender=self.__class__, instance=duplicate)

sample/models.py

@@ -107,11 +107,6 @@ class Book(CloneModel):
     def __str__(self):
         return _(self.name)
 
-    def pre_save_duplicate(self, instance):
-        instance.name = self.name
-
-        return instance
-
 
 class BookTag(models.Model):
     book = models.ForeignKey(Book, on_delete=models.CASCADE)