Rename Iterable due to typing.Iterable. Add deprecation warning

Author: Yobmod

git/util.py

@@ -18,6 +18,7 @@
 import time
 from unittest import SkipTest
 from urllib.parse import urlsplit, urlunsplit
+import warnings
 
 # typing ---------------------------------------------------------
 
@@ -1013,15 +1014,52 @@ def __delitem__(self, index: Union[SupportsIndex, int, slice, str]) -> Any:
         list.__delitem__(self, delindex)
 
 
+class IterableClassWatcher(type):
+    def __init__(cls, name, bases, clsdict):
+        for base in bases:
+            if type(base) == cls:
+                warnings.warn("GitPython Iterable is deprecated due to naming clash. Use IterableObj instead",
+                              DeprecationWarning)
+        super(IterableClassWatcher, cls).__init__(name, bases, clsdict)
+
+
 class Iterable(object):
 
     """Defines an interface for iterable items which is to assure a uniform
     way to retrieve and iterate items within the git repository"""
     __slots__ = ()
     _id_attribute_ = "attribute that most suitably identifies your instance"
+    __metaclass__ = IterableClassWatcher
 
     @classmethod
-    def list_items(cls, repo: 'Repo', *args: Any, **kwargs: Any) -> IterableList['IterableObj']:
+    def list_items(cls, repo, *args, **kwargs):
+        """
+        Find all items of this type - subclasses can specify args and kwargs differently.
+        If no args are given, subclasses are obliged to return all items if no additional
+        arguments arg given.
+
+        :note: Favor the iter_items method as it will
+        :return:list(Item,...) list of item instances"""
+        out_list = IterableList(cls._id_attribute_)
+        out_list.extend(cls.iter_items(repo, *args, **kwargs))
+        return out_list
+
+    @classmethod
+    def iter_items(cls, repo: 'Repo', *args: Any, **kwargs: Any):
+        # return typed to be compatible with subtypes e.g. Remote
+        """For more information about the arguments, see list_items
+        :return:  iterator yielding Items"""
+        raise NotImplementedError("To be implemented by Subclass")
+
+
+class IterableObj():
+    """Defines an interface for iterable items which is to assure a uniform
+    way to retrieve and iterate items within the git repository"""
+    __slots__ = ()
+    _id_attribute_ = "attribute that most suitably identifies your instance"
+
+    @classmethod
+    def list_items(cls, repo: 'Repo', *args: Any, **kwargs: Any) -> IterableList[T]:
         """
         Find all items of this type - subclasses can specify args and kwargs differently.
         If no args are given, subclasses are obliged to return all items if no additional
@@ -1044,10 +1082,6 @@ def iter_items(cls, repo: 'Repo', *args: Any, **kwargs: Any) -> Iterator:
 #} END classes
 
 
-class IterableObj(Iterable):
-    pass
-
-
 class NullHandler(logging.Handler):
     def emit(self, record: object) -> None:
         pass

t.py

@@ -0,0 +1,19 @@
+class Watcher(type):
+    def __init__(cls, name, bases, clsdict):
+        [print("ooooo") for base in bases if issubclass(base, name)]
+        super(Watcher, cls).__init__(name, bases, clsdict)
+
+
+class SuperClass(metaclass=Watcher):
+    pass
+
+
+class SubClass0(SuperClass):
+    pass
+
+
+class SubClass1(SuperClass):
+    print("test")
+
+class normo():
+    print("wooo")