fix: cleanup commented methods

Author: lwasser

src/pyosmeta/github_api.py

@@ -185,53 +185,6 @@ def _get_response_rest(self, url: str) -> list[dict[str, Any]]:
 
         return results
 
-    # def return_response(self) -> list[dict[str, object]]:
-    #     """TODO: delete this one all works
-    #     Make a GET request to the Github API endpoint
-    #     Deserialize json response to list of dicts.
-
-    #     Handles pagination as github has a REST api 100 request max.
-
-    #     Returns
-    #     -------
-    #     list
-    #         List of dict items each containing a review issue
-    #     """
-
-    #     results = []
-    #     # This is computed as a property. Reassign here to support pagination
-    #     # and new urls for each page
-    #     api_endpoint_url = self.api_endpoint
-    #     try:
-    #         while True:
-    #             response = requests.get(
-    #                 api_endpoint_url,
-    #                 headers={"Authorization": f"token {self.get_token()}"},
-    #             )
-    #             response.raise_for_status()
-    #             results.extend(response.json())
-
-    #             # Check if there are more pages to fetch
-    #             if "next" in response.links:
-    #                 next_url = response.links["next"]["url"]
-    #                 api_endpoint_url = next_url
-    #             else:
-    #                 break
-
-    #             # Handle rate limiting
-    #             self.handle_rate_limit(response)
-
-    #     except requests.HTTPError as exception:
-    #         if exception.response.status_code == 401:
-    #             print(
-    #                 "Oops - your request isn't authorized. Your token may be "
-    #                 "expired or invalid. Please refresh your token."
-    #             )
-    #         else:
-    #             raise exception
-
-    #     return results
-
     def get_gh_metrics(
         self,
         endpoints: dict[dict[str, str]],

src/pyosmeta/parse_issues.py

@@ -61,10 +61,10 @@ def get_issues(self) -> list[Issue]:
         grabbing issues with ALL using an and operator labels in a list. We
         need to use an OR as a selector.
         """
-        # TODO:
+
         url = self.github_api.api_endpoint
         issues = self.github_api._get_response_rest(url)
-        # issues = self.github_api.return_response()
+
         # Filter issues according to label query value
         labels = self.github_api.labels
         filtered_issues = [
@@ -321,9 +321,6 @@ def parse_issues(
         errors = {}
         for issue in issues:
             print(f"Processing review {issue.title}")
-            # if "Stingray" in issue.title:
-            #     print("Stop now!")
-            #     break
 
             try:
                 review = self.parse_issue(issue)
@@ -396,8 +393,6 @@ def get_repo_paths(
             all_repos[a_package] = {"owner": owner, "repo_name": repo}
         return all_repos
 
-    # This works - i could just make it more generic and remove fmt since it's
-    # not used and replace it with a number of values and a test string
     def get_categories(
         self,
         issue_list: list[str],