From 5f4b4dbff46fae4c899f5573aea5a7266a41eeeb Mon Sep 17 00:00:00 2001 From: Russ Allbery Date: Mon, 20 Sep 2021 13:53:42 -0700 Subject: Fix typing issues with delete_head and Remote.add delete_head and Head.delete historically accept either Head objects or a str name of a head. Adjust the typing to match. This unfortunately requires suppressing type warnings in the signature of RemoteReference.delete, since it inherits from Head but does not accept str (since it needs access to the richer data of RemoteReference). Using assignment to make add an alias for create unfortunately confuses mypy, since it loses track of the fact that it's a classmethod and starts treating it like a staticmethod. Replace with a stub wrapper instead. --- git/remote.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'git/remote.py') diff --git a/git/remote.py b/git/remote.py index 9917c431..2cf5678b 100644 --- a/git/remote.py +++ b/git/remote.py @@ -665,7 +665,9 @@ class Remote(LazyMixin, IterableObj): 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: -- cgit v1.2.1