Skip to content

Commit

Permalink
remove unused variable
Browse files Browse the repository at this point in the history
Signed-off-by: Isaac Milarsky <[email protected]>
  • Loading branch information
IsaacMilarky committed Jan 23, 2024
1 parent fc3d13b commit 579dec0
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions augur/tasks/gitlab/merge_request_task.py
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 579dec0

Please sign in to comment.