Skip to content

Commit

Permalink
use vars
Browse files Browse the repository at this point in the history
Signed-off-by: Isaac Milarsky <isaac.milarsky@hhs.cms.gov>
  • Loading branch information
IsaacMilarky committed Jan 23, 2024
1 parent 579dec0 commit 59afa43
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions augur/tasks/gitlab/merge_request_task.py
Original file line number Diff line number Diff line change
Expand Up @@ -346,11 +346,11 @@ 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, repo_id, logger, augur_db)
process_mr_reviewers(reviewers, f"{owner}/{repo}: Mr reviewer task", repo_id, logger, augur_db)
else:
logger.info(f"{owner}/{repo} has no gitlab merge request reviewers")

def process_mr_reviewers(data, repo_id, logger, augur_db):
def process_mr_reviewers(data, task_name, repo_id, logger, augur_db):
"""
Retrieve only the needed data for mr reviewer data from the api response
Expand All @@ -365,6 +365,8 @@ def process_mr_reviewers(data, repo_id, logger, augur_db):
tool_version = "2.0"
data_source = "Gitlab API"

logger.info(f"Running {task_name}...")

# create mapping from mr number to pull request id of current mrs
mr_number_to_id_map = {}
mrs = augur_db.session.query(PullRequest).filter(PullRequest.repo_id == repo_id).all()
Expand Down

0 comments on commit 59afa43

Please sign in to comment.