Do not push branch if the aider invocation fails #28

Closed
Jmaa wants to merge 2 commits from jmaa/issue-23-do-not-push-branch-if-the-aider-invocation-fails into main
Showing only changes of commit 78c995c1a7 - Show all commits

View File

@ -211,7 +211,9 @@ def has_commits_on_branch(cwd: Path, base_branch: str, current_branch: str) -> b
return False return False
def run_cmd(cmd: list[str], cwd: Path | None = None, check=True) -> subprocess.CompletedProcess: def run_cmd(
cmd: list[str], cwd: Path | None = None, check=True,
) -> subprocess.CompletedProcess:
print(cmd) print(cmd)
return subprocess.run(cmd, check=check, cwd=cwd) return subprocess.run(cmd, check=check, cwd=cwd)
@ -237,12 +239,16 @@ def solve_issue_in_repository(
# Run aider # Run aider
try: try:
aider_result = run_cmd(create_aider_command(f'# {issue_title}\n{issue_description}'), tmpdirname) aider_result = run_cmd(
create_aider_command(f'# {issue_title}\n{issue_description}'), tmpdirname,
)
if aider_result.returncode != 0: if aider_result.returncode != 0:
logger.error(f"Aider invocation failed for issue #{issue_number}") logger.error(f'Aider invocation failed for issue #{issue_number}')
return False return False
except subprocess.CalledProcessError: except subprocess.CalledProcessError:
logger.exception(f"Aider invocation failed with an exception for issue #{issue_number}") logger.exception(
f'Aider invocation failed with an exception for issue #{issue_number}',
)
return False return False
# Auto-fix standard code quality stuff # Auto-fix standard code quality stuff