From 7104eb078f100eb65641af086088a0b5c5841bf6 Mon Sep 17 00:00:00 2001 From: dw-0 Date: Sun, 24 Mar 2024 00:02:10 +0100 Subject: [PATCH] refactor(RepoManager): if no branch is given, no checkout is done Signed-off-by: Dominik Willner --- kiauh/core/repo_manager/repo_manager.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/kiauh/core/repo_manager/repo_manager.py b/kiauh/core/repo_manager/repo_manager.py index 6a7cfa0..956d6ec 100644 --- a/kiauh/core/repo_manager/repo_manager.py +++ b/kiauh/core/repo_manager/repo_manager.py @@ -24,7 +24,7 @@ class RepoManager: branch: str = None, ): self._repo = repo - self._branch = branch if branch is not None else "master" + self._branch = branch self._method = self._get_method() self._target_dir = target_dir @@ -110,7 +110,7 @@ class RepoManager: if Path(self.target_dir).exists(): question = f"'{self.target_dir}' already exists. Overwrite?" if not get_confirm(question, default_choice=False): - Logger.print_info("Skipping re-clone of repository.") + Logger.print_info("Skip cloning of repository ...") return shutil.rmtree(self.target_dir) @@ -145,6 +145,9 @@ class RepoManager: raise def _checkout(self): + if self.branch is None: + return + try: command = ["git", "checkout", f"{self.branch}"] subprocess.run(command, cwd=self.target_dir, check=True)