Add friendlier error handling for git repository cloning errors #2

New issue

Open

opened 2026-03-28 22:38:30 -04:00 by danielfdickinson · 0 comments

danielfdickinson commented

2026-03-28 22:38:30 -04:00

Owner

Description: Add explicit error handling around git.Repo.clone_from() to provide better visibility into cloning failures while maintaining fatal error behavior.

Current behavior: The git.Repo.clone_from() call in src/repotool/repos.py can fail and raise exceptions without specific logging.

Proposed enhancement:

From GitLabDuo - review and adjust as appropriate.

try:
  new_repo = git.Repo.clone_from(repo_url, repo_path)
except Exception as e:
  logger.error("Failed to clone repository from %s: %s", repo_url, str(e))
  raise

This will:

Log specific error details for debugging
Re-raise exceptions to maintain fatal behavior
Prevent RepoUpdater creation when cloning fails
Provide better visibility into failures

**Description:** Add explicit error handling around `git.Repo.clone_from()` to provide better visibility into cloning failures while maintaining fatal error behavior. **Current behavior:** The `git.Repo.clone_from()` call in `src/repotool/repos.py` can fail and raise exceptions without specific logging. **Proposed enhancement:** From GitLabDuo - review and adjust as appropriate. ```python try: new_repo = git.Repo.clone_from(repo_url, repo_path) except Exception as e: logger.error("Failed to clone repository from %s: %s", repo_url, str(e)) raise ``` This will: * Log specific error details for debugging * Re-raise exceptions to maintain fatal behavior * Prevent RepoUpdater creation when cloning fails * Provide better visibility into failures