From 579dec0d90cb8e38cd72fd6b2f02dc4659bdd624 Mon Sep 17 00:00:00 2001 From: Isaac Milarsky Date: Tue, 23 Jan 2024 12:26:38 -0600 Subject: [PATCH] remove unused variable Signed-off-by: Isaac Milarsky --- augur/tasks/gitlab/merge_request_task.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/augur/tasks/gitlab/merge_request_task.py b/augur/tasks/gitlab/merge_request_task.py index a04f055a27..60d2ececda 100644 --- a/augur/tasks/gitlab/merge_request_task.py +++ b/augur/tasks/gitlab/merge_request_task.py @@ -346,17 +346,16 @@ def collect_merge_request_reviewers(mr_ids, repo_git) -> int: if reviewers: logger.info(f"Length of merge request reviewers: {len(reviewers)}") - process_mr_reviewers(reviewers, f"{owner}/{repo}: Mr reviewer task", repo_id, logger, augur_db) + process_mr_reviewers(reviewers, repo_id, logger, augur_db) else: logger.info(f"{owner}/{repo} has no gitlab merge request reviewers") -def process_mr_reviewers(data, task_name, repo_id, logger, augur_db): +def process_mr_reviewers(data, repo_id, logger, augur_db): """ Retrieve only the needed data for mr reviewer data from the api response Arguments: data: List of dictionaries of mr reviewer data - task_name: name of the task as well as the repo being processed repo_id: augur id of the repo logger: logging object augur_db: sqlalchemy db object