fix + docstring

collectfast/strategies/base.py

@@ -15,7 +15,6 @@
 from django.core.cache import caches
 from django.core.files.storage import Storage
 from django.utils.encoding import force_bytes
-from storages.utils import safe_join
 from typing_extensions import ClassVar
 
 from collectfast import settings
@@ -36,6 +35,12 @@ def __init__(self, remote_storage):
     @abc.abstractmethod
     def should_copy_file(self, path, prefixed_path, local_storage):
         # type: (str, str, Storage) -> bool
+        """
+        Called for each file before copying happens, this method decides
+        whether a file should be copied or not. Return False to indicate that
+        the file is already up-to-date and should not be copied, or True to
+        indicate that it is stale and needs updating.
+        """
         ...
 
     def pre_should_copy_hook(self):

collectfast/tests/test_command.py

@@ -1,5 +1,3 @@
-import warnings
-
 from django.core.management import call_command
 from django.test import override_settings as override_django_settings
 from django.utils.six import StringIO