diff --git a/pr_agent/git_providers/azuredevops_provider.py b/pr_agent/git_providers/azuredevops_provider.py index 197b47c5..2e1c57b6 100644 --- a/pr_agent/git_providers/azuredevops_provider.py +++ b/pr_agent/git_providers/azuredevops_provider.py @@ -623,6 +623,3 @@ class AzureDevopsProvider(GitProvider): def get_line_link(self, relevant_file: str, relevant_line_start: int, relevant_line_end: int = None) -> str: return self.pr_url+f"?_a=files&path={relevant_file}" - - def _prepare_clone_url_with_token(self, repo_url_to_clone: str) -> str | None: - raise Exception("Not implemented!") diff --git a/pr_agent/git_providers/codecommit_provider.py b/pr_agent/git_providers/codecommit_provider.py index 3dbe15bc..c4f1ed7b 100644 --- a/pr_agent/git_providers/codecommit_provider.py +++ b/pr_agent/git_providers/codecommit_provider.py @@ -495,6 +495,3 @@ class CodeCommitProvider(GitProvider): lang: round(count / total_files * 100) for lang, count in lang_count.items() } return lang_percentage - - def _prepare_clone_url_with_token(self, repo_url_to_clone: str) -> str | None: - raise Exception("Not implemented!") diff --git a/pr_agent/git_providers/gerrit_provider.py b/pr_agent/git_providers/gerrit_provider.py index cce4fad4..ba587656 100644 --- a/pr_agent/git_providers/gerrit_provider.py +++ b/pr_agent/git_providers/gerrit_provider.py @@ -397,6 +397,3 @@ class GerritProvider(GitProvider): def get_pr_branch(self): return self.repo.head - - def _prepare_clone_url_with_token(self, repo_url_to_clone: str) -> str | None: - raise Exception("Not implemented!") diff --git a/pr_agent/git_providers/git_provider.py b/pr_agent/git_providers/git_provider.py index 4aecbf0d..e5afe42b 100644 --- a/pr_agent/git_providers/git_provider.py +++ b/pr_agent/git_providers/git_provider.py @@ -46,9 +46,10 @@ class GitProvider(ABC): shutil.rmtree(self.path, ignore_errors=True) @abstractmethod - #Method to allow implementors to manipulate the repo url to clone (such as embedding tokens in the url string). + #Method to allow implementors to manipulate the repo url to clone (such as embedding tokens in the url string). Needs to be implemented by the provider. def _prepare_clone_url_with_token(self, repo_url_to_clone: str) -> str | None: - pass + get_logger().warning("Not implemented! Returning None") + return None # Does a shallow clone, using a forked process to support a timeout guard. # In case operation has failed, it is expected to throw an exception as this method does not return a value. diff --git a/pr_agent/git_providers/local_git_provider.py b/pr_agent/git_providers/local_git_provider.py index beaab24d..42028976 100644 --- a/pr_agent/git_providers/local_git_provider.py +++ b/pr_agent/git_providers/local_git_provider.py @@ -190,6 +190,3 @@ class LocalGitProvider(GitProvider): def get_pr_labels(self, update=False): raise NotImplementedError('Getting labels is not implemented for the local git provider') - - def _prepare_clone_url_with_token(self, repo_url_to_clone: str) -> str | None: - raise Exception("Not implemented!")