pickman: Disable remaining pylint warnings

Add pylint disable comments for too-many-arguments, too-many-locals,
and too-many-branches warnings on functions that necessarily have
complex logic due to the nature of building agent prompts and
processing MR reviews.

Co-developed-by: Claude <noreply@anthropic.com>
This commit is contained in:
Simon Glass
2025-12-20 11:42:01 -07:00
parent 3e62b6ee5c
commit ed3328458d
2 changed files with 5 additions and 0 deletions

View File

@@ -185,6 +185,7 @@ Rebase instructions:
return context_section, comment_section, rebase_section
# pylint: disable=too-many-arguments
def build_review_prompt(mr_iid, branch_name, task_desc, context_section,
comment_section, rebase_section, comments,
needs_rebase, remote, target):
@@ -246,6 +247,7 @@ Important:
"""
# pylint: disable=too-many-arguments
def build_full_review_prompt(mr_iid, branch_name, comments, remote, target,
needs_rebase, has_conflicts, mr_description):
"""Build complete prompt and task description for the review agent
@@ -287,6 +289,7 @@ def build_full_review_prompt(mr_iid, branch_name, comments, remote, target,
return prompt, task_desc
# pylint: disable=too-many-arguments,too-many-locals
async def run_review_agent(mr_iid, branch_name, comments, remote,
target='master', needs_rebase=False,
has_conflicts=False, mr_description='',
@@ -339,6 +342,7 @@ async def run_review_agent(mr_iid, branch_name, comments, remote,
return False, '\n\n'.join(conversation_log)
# pylint: disable=too-many-arguments
def handle_mr_comments(mr_iid, branch_name, comments, remote, target='master',
needs_rebase=False, has_conflicts=False,
mr_description='', repo_path=None):

View File

@@ -648,6 +648,7 @@ def do_commit_source(args, dbs):
return 0
# pylint: disable=too-many-locals,too-many-branches
def process_mr_reviews(remote, mrs, dbs, target='master'):
"""Process review comments on open MRs