Add types to Remote. create() remove() rename() and make agree with base.Repo

Author: Yobmod

git/remote.py

@@ -625,7 +625,7 @@ def stale_refs(self) -> IterableList:
         return out_refs
 
     @classmethod
-    def create(cls, repo, name, url, **kwargs):
+    def create(cls, repo: 'Repo', name: str, url: str, **kwargs: Any) -> 'Remote':
         """Create a new remote to the given repository
         :param repo: Repository instance that is to receive the new remote
         :param name: Desired name of the remote
@@ -642,7 +642,7 @@ def create(cls, repo, name, url, **kwargs):
     add = create
 
     @classmethod
-    def remove(cls, repo, name):
+    def remove(cls, repo: 'Repo', name: str) -> str:
         """Remove the remote with the given name
         :return: the passed remote name to remove
         """
@@ -654,7 +654,7 @@ def remove(cls, repo, name):
     # alias
     rm = remove
 
-    def rename(self, new_name):
+    def rename(self, new_name: str) -> 'Remote':
         """Rename self to the given new_name
         :return: self """
         if self.name == new_name:

git/repo/base.py

@@ -4,6 +4,7 @@
 # This module is part of GitPython and is released under
 # the BSD License: http://www.opensource.org/licenses/bsd-license.php
 
+from _typeshed import StrPath
 import logging
 import os
 import re
@@ -434,7 +435,7 @@ def delete_tag(self, *tags: TBD) -> None:
         """Delete the given tag references"""
         return TagReference.delete(self, *tags)
 
-    def create_remote(self, name: str, url: PathLike, **kwargs: Any) -> Remote:
+    def create_remote(self, name: str, url: str, **kwargs: Any) -> Remote:
         """Create a new remote.
 
         For more information, please see the documentation of the Remote.create
@@ -443,7 +444,7 @@ def create_remote(self, name: str, url: PathLike, **kwargs: Any) -> Remote:
         :return: Remote reference"""
         return Remote.create(self, name, url, **kwargs)
 
-    def delete_remote(self, remote: 'Remote') -> Type['Remote']:
+    def delete_remote(self, remote: 'Remote') -> str:
         """Delete the given remote."""
         return Remote.remove(self, remote)