gitpython-developers · Yobmod · Sep 24, 2021 · Sep 20, 2021
diff --git a/git/refs/head.py b/git/refs/head.py
@@ -129,7 +129,7 @@ class Head(Reference):
    k_config_remote_ref = "merge"           # branch to merge from remote

    @classmethod
-    def delete(cls, repo: 'Repo', *heads: 'Head', force: bool = False, **kwargs: Any) -> None:
+    def delete(cls, repo: 'Repo', *heads: 'Union[Head, str]', force: bool = False, **kwargs: Any) -> None:
        """Delete the given heads

        :param force:

diff --git a/git/refs/remote.py b/git/refs/remote.py
@@ -37,8 +37,13 @@ def iter_items(cls, repo: 'Repo', common_path: Union[PathLike, None] = None,
        # super is Reference
        return super(RemoteReference, cls).iter_items(repo, common_path)

+    # The Head implementation of delete also accepts strs, but this
+    # implementation does not.  mypy doesn't have a way of representing
+    # tightening the types of arguments in subclasses and recommends Any or
+    # "type: ignore".  (See https://github.com/python/typing/issues/241)
    @ classmethod
-    def delete(cls, repo: 'Repo', *refs: 'RemoteReference', **kwargs: Any) -> None:
+    def delete(cls, repo: 'Repo', *refs: 'RemoteReference',  # type: ignore
+               **kwargs: Any) -> None:
        """Delete the given remote references

        :note:

diff --git a/git/remote.py b/git/remote.py
@@ -665,7 +665,9 @@ def create(cls, repo: 'Repo', name: str, url: str, **kwargs: Any) -> 'Remote':
        return cls(repo, name)

    # add is an alias
-    add = create
+    @ classmethod
+    def add(cls, repo: 'Repo', name: str, url: str, **kwargs: Any) -> 'Remote':
+        return cls.create(repo, name, url, **kwargs)

    @ classmethod
    def remove(cls, repo: 'Repo', name: str) -> str:

diff --git a/git/repo/base.py b/git/repo/base.py
@@ -429,7 +429,7 @@ def create_head(self, path: PathLike, commit: str = 'HEAD',
        :return: newly created Head Reference"""
        return Head.create(self, path, commit, logmsg, force)

-    def delete_head(self, *heads: 'Head', **kwargs: Any) -> None:
+    def delete_head(self, *heads: 'Union[str, Head]', **kwargs: Any) -> None:
        """Delete the given heads

        :param kwargs: Additional keyword arguments to be passed to git-branch"""