From e9d479c32cf8293d2f94564563057c3408ce373f Mon Sep 17 00:00:00 2001 From: Leah Wasser Date: Wed, 28 Feb 2024 20:05:39 -0700 Subject: [PATCH] Fix: remove commented out code --- src/pyosmeta/parse_issues.py | 21 +-------------------- 1 file changed, 1 insertion(+), 20 deletions(-) diff --git a/src/pyosmeta/parse_issues.py b/src/pyosmeta/parse_issues.py index 061cac8..6e07909 100644 --- a/src/pyosmeta/parse_issues.py +++ b/src/pyosmeta/parse_issues.py @@ -622,28 +622,12 @@ def get_issue_meta( issue_meta = {} # TODO: change to for line in review_comment for single_line in body_data[0:end_range]: - # TODO - i think this will be easier to read if the code to parse - # each line is here rather than another redirect. - # Fix: this method self.get_line_meta is what i'm removing - # issue_meta.update(self._get_line_meta(item)) - meta = {} a_key = single_line[0].lower().replace(" ", "_") # If the line is for a review role - editor, maintainer, reviewer if self._is_review_role(single_line[0]): + # Collect metadata for each review role meta = self.get_contributor_data(single_line) - # # Parse comma separated names for maintainer list - # if single_line[0].startswith("All current maintainers"): - # names = single_line[1].split(",") - # meta[a_key] = [] - # for name in names: - # # Add each maintainer to the dict - # a_maint = parse_user_names(username=name) - # meta[a_key].append(a_maint) - # # Parse other review roles; these have one name per line - # else: - # names = parse_user_names(single_line[1]) - # meta[a_key] = names elif len(single_line) > 1: meta[a_key] = single_line[1].strip() else: @@ -789,9 +773,6 @@ def get_repo_meta(self, url: str, stats_list: list) -> dict: for astat in stats_list: stats_dict[astat] = data[astat] stats_dict["documentation"] = stats_dict.pop("homepage") - # stats_dict["created_at"] = self._clean_date( - # stats_dict["created_at"] - # ) return stats_dict