Ruff after Claude Code
This commit is contained in:
parent
9675123a5b
commit
7145d4f72e
|
@ -156,6 +156,8 @@ class RepositoryConfig:
|
||||||
owner: str
|
owner: str
|
||||||
repo: str
|
repo: str
|
||||||
base_branch: str
|
base_branch: str
|
||||||
|
assignee: str | None = None
|
||||||
|
reviewer: str | None = None
|
||||||
|
|
||||||
def repo_url(self) -> str:
|
def repo_url(self) -> str:
|
||||||
return f'{self.gitea_url}:{self.owner}/{self.repo}.git'.replace(
|
return f'{self.gitea_url}:{self.owner}/{self.repo}.git'.replace(
|
||||||
|
@ -506,6 +508,8 @@ def push_changes(
|
||||||
head=branch_name,
|
head=branch_name,
|
||||||
base=repository_config.base_branch,
|
base=repository_config.base_branch,
|
||||||
labels=['aider'],
|
labels=['aider'],
|
||||||
|
assignee=repository_config.assignee,
|
||||||
|
reviewer=repository_config.reviewer,
|
||||||
)
|
)
|
||||||
|
|
||||||
# Extract PR number and URL if available
|
# Extract PR number and URL if available
|
||||||
|
|
|
@ -55,6 +55,14 @@ def parse_args():
|
||||||
help='Model to use for evaluating code (overrides default)',
|
help='Model to use for evaluating code (overrides default)',
|
||||||
default=None,
|
default=None,
|
||||||
)
|
)
|
||||||
|
parser.add_argument(
|
||||||
|
'--assignee',
|
||||||
|
help='Username to assign as the assignee for created pull requests',
|
||||||
|
)
|
||||||
|
parser.add_argument(
|
||||||
|
'--reviewer',
|
||||||
|
help='Username to assign as the reviewer for created pull requests',
|
||||||
|
)
|
||||||
return parser.parse_args()
|
return parser.parse_args()
|
||||||
|
|
||||||
|
|
||||||
|
@ -84,6 +92,8 @@ def main():
|
||||||
owner=args.owner,
|
owner=args.owner,
|
||||||
repo=repo,
|
repo=repo,
|
||||||
base_branch=args.base_branch,
|
base_branch=args.base_branch,
|
||||||
|
assignee=args.assignee,
|
||||||
|
reviewer=args.reviewer,
|
||||||
)
|
)
|
||||||
solve_issues_in_repository(repository_config, client, seen_issues_db)
|
solve_issues_in_repository(repository_config, client, seen_issues_db)
|
||||||
del repo
|
del repo
|
||||||
|
|
|
@ -139,8 +139,10 @@ class GiteaClient:
|
||||||
head: str,
|
head: str,
|
||||||
base: str,
|
base: str,
|
||||||
labels: list[str] = None,
|
labels: list[str] = None,
|
||||||
|
assignee: str | None = None,
|
||||||
|
reviewer: str | None = None,
|
||||||
) -> dict:
|
) -> dict:
|
||||||
"""Create a pull request and optionally apply labels.
|
"""Create a pull request and optionally apply labels, assignee, and reviewer.
|
||||||
|
|
||||||
Args:
|
Args:
|
||||||
owner (str): Owner of the repository.
|
owner (str): Owner of the repository.
|
||||||
|
@ -150,6 +152,8 @@ class GiteaClient:
|
||||||
head (str): The name of the branch where changes are implemented.
|
head (str): The name of the branch where changes are implemented.
|
||||||
base (str): The name of the branch you want the changes pulled into.
|
base (str): The name of the branch you want the changes pulled into.
|
||||||
labels (list[str], optional): List of label names to apply to the pull request.
|
labels (list[str], optional): List of label names to apply to the pull request.
|
||||||
|
assignee (str, optional): Username to assign as the assignee.
|
||||||
|
reviewer (str, optional): Username to assign as the reviewer.
|
||||||
|
|
||||||
Returns:
|
Returns:
|
||||||
dict: The created pull request data.
|
dict: The created pull request data.
|
||||||
|
@ -183,7 +187,75 @@ class GiteaClient:
|
||||||
# fallback to raise if we can’t find it
|
# fallback to raise if we can’t find it
|
||||||
response.raise_for_status()
|
response.raise_for_status()
|
||||||
response.raise_for_status()
|
response.raise_for_status()
|
||||||
return response.json()
|
|
||||||
|
pr_data = response.json()
|
||||||
|
pr_number = pr_data.get('number')
|
||||||
|
|
||||||
|
# Assign assignee if specified
|
||||||
|
if assignee and pr_number:
|
||||||
|
self._assign_to_pull_request(owner, repo, pr_number, assignee)
|
||||||
|
|
||||||
|
# Assign reviewer if specified
|
||||||
|
if reviewer and pr_number:
|
||||||
|
self._assign_reviewer_to_pull_request(owner, repo, pr_number, reviewer)
|
||||||
|
|
||||||
|
return pr_data
|
||||||
|
|
||||||
|
def _assign_to_pull_request(
|
||||||
|
self,
|
||||||
|
owner: str,
|
||||||
|
repo: str,
|
||||||
|
pr_number: int,
|
||||||
|
assignee: str,
|
||||||
|
) -> None:
|
||||||
|
"""Assign a user to a pull request.
|
||||||
|
|
||||||
|
Args:
|
||||||
|
owner (str): Owner of the repository.
|
||||||
|
repo (str): Name of the repository.
|
||||||
|
pr_number (int): Pull request number.
|
||||||
|
assignee (str): Username to assign.
|
||||||
|
"""
|
||||||
|
url = f'{self.gitea_url}/repos/{owner}/{repo}/issues/{pr_number}/assignees'
|
||||||
|
json_data = {'assignees': [assignee]}
|
||||||
|
response = self.session.post(url, json=json_data)
|
||||||
|
if response.status_code != 201:
|
||||||
|
logger.warning(
|
||||||
|
'Failed to assign user %s to PR #%s: %s',
|
||||||
|
assignee,
|
||||||
|
pr_number,
|
||||||
|
response.text,
|
||||||
|
)
|
||||||
|
else:
|
||||||
|
logger.info('Assigned user %s to PR #%s', assignee, pr_number)
|
||||||
|
|
||||||
|
def _assign_reviewer_to_pull_request(
|
||||||
|
self,
|
||||||
|
owner: str,
|
||||||
|
repo: str,
|
||||||
|
pr_number: int,
|
||||||
|
reviewer: str,
|
||||||
|
) -> None:
|
||||||
|
"""Assign a reviewer to a pull request.
|
||||||
|
|
||||||
|
Args:
|
||||||
|
owner (str): Owner of the repository.
|
||||||
|
repo (str): Name of the repository.
|
||||||
|
pr_number (int): Pull request number.
|
||||||
|
reviewer (str): Username to assign as reviewer.
|
||||||
|
"""
|
||||||
|
url = f'{self.gitea_url}/repos/{owner}/{repo}/pulls/{pr_number}/requested_reviewers'
|
||||||
|
json_data = {'reviewers': [reviewer]}
|
||||||
|
response = self.session.post(url, json=json_data)
|
||||||
|
if response.status_code != 201:
|
||||||
|
logger.warning(
|
||||||
|
'Failed to assign reviewer %s to PR #%s: %s',
|
||||||
|
reviewer,
|
||||||
|
pr_number,
|
||||||
|
response.text,
|
||||||
|
)
|
||||||
|
else:
|
||||||
|
logger.info('Assigned reviewer %s to PR #%s', reviewer, pr_number)
|
||||||
|
|
||||||
def get_failed_pipelines(self, owner: str, repo: str, pr_number: str) -> list[int]:
|
def get_failed_pipelines(self, owner: str, repo: str, pr_number: str) -> list[int]:
|
||||||
"""Fetch pipeline runs for a PR and return IDs of failed runs."""
|
"""Fetch pipeline runs for a PR and return IDs of failed runs."""
|
||||||
|
|
Loading…
Reference in New Issue
Block a user