Skip to content

Commit

Permalink
Fix: remove commented out code
Browse files Browse the repository at this point in the history
  • Loading branch information
lwasser committed Feb 29, 2024
1 parent 9fc98f7 commit e9d479c
Showing 1 changed file with 1 addition and 20 deletions.
21 changes: 1 addition & 20 deletions src/pyosmeta/parse_issues.py
Original file line number Diff line number Diff line change
Expand Up @@ -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(" ", "_")

Check warning on line 626 in src/pyosmeta/parse_issues.py

View check run for this annotation

Codecov / codecov/patch

src/pyosmeta/parse_issues.py#L625-L626

Added lines #L625 - L626 were not covered by tests
# 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)

Check warning on line 630 in src/pyosmeta/parse_issues.py

View check run for this annotation

Codecov / codecov/patch

src/pyosmeta/parse_issues.py#L630

Added line #L630 was not covered by tests
# # 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()

Check warning on line 632 in src/pyosmeta/parse_issues.py

View check run for this annotation

Codecov / codecov/patch

src/pyosmeta/parse_issues.py#L632

Added line #L632 was not covered by tests
else:
Expand Down Expand Up @@ -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

Expand Down

0 comments on commit e9d479c

Please sign in to comment.