chore: fix spelling and formatting

Signed-off-by: Zack Koppert <zkoppert@github.com>

Author: zkoppert

auth.py

@@ -41,10 +41,8 @@ def auth_to_github(
     elif token:
         github_connection = github3.login(token=token)
     else:
-        raise ValueError(
-            "GH_TOKEN or the set of [GH_APP_ID, GH_APP_INSTALLATION_ID, \
-                GH_APP_PRIVATE_KEY] environment variables are not set"
-        )
+        raise ValueError("GH_TOKEN or the set of [GH_APP_ID, GH_APP_INSTALLATION_ID, \
+                GH_APP_PRIVATE_KEY] environment variables are not set")
 
     if not github_connection:
         raise ValueError("Unable to authenticate to GitHub")

markdown_writer.py

@@ -159,10 +159,8 @@ def write_to_markdown(
         # If all the metrics are None, then there are no issues
         if not issues_with_metrics or len(issues_with_metrics) == 0:
             file.write("no issues found for the given search criteria\n\n")
-            file.write(
-                "\n_This report was generated with the \
-[Issue Metrics Action](https://github.com/github/issue-metrics)_\n"
-            )
+            file.write("\n_This report was generated with the \
+[Issue Metrics Action](https://github.com/github/issue-metrics)_\n")
             if search_query:
                 file.write(f"Search query used to find these items: `{search_query}`\n")
             return
@@ -254,10 +252,8 @@ def write_to_markdown(
                     file.write(f" {issue.pr_comment_count or 'N/A'} |")
                 file.write("\n")
             file.write("\n")
-        file.write(
-            "_This report was generated with the \
-[Issue Metrics Action](https://github.com/github/issue-metrics)_\n"
-        )
+        file.write("_This report was generated with the \
+[Issue Metrics Action](https://github.com/github/issue-metrics)_\n")
         if search_query:
             file.write(f"Search query used to find these items: `{search_query}`\n")
 

most_active_mentors.py

@@ -13,7 +13,7 @@
    - should there be a limit to how many comments per PR we consider to avoid
      having the statistic dominated by contested PRs?
    - should this metric count consecutive comments coming from the same user as
-     only one to avoid people unnessesarily splitting their comments to game the
+     only one to avoid people unnecessarily splitting their comments to game the
      metric?
    - instead of PR comments should we count PRs on which a username was seen as
      commenter?

search.py

@@ -81,17 +81,13 @@ def wait_for_api_refresh(
                 wait_for_api_refresh(issues_iterator, rate_limit_bypass)
 
     except github3.exceptions.ForbiddenError as e:
-        print(
-            f"You do not have permission to view a repository \
-from: '{repos_and_owners_string}'; Check your API Token."
-        )
+        print(f"You do not have permission to view a repository \
+from: '{repos_and_owners_string}'; Check your API Token.")
         print_error_messages(e)
         sys.exit(1)
     except github3.exceptions.NotFoundError as e:
-        print(
-            f"The repository could not be found; \
-Check the repository owner and names: '{repos_and_owners_string}"
-        )
+        print(f"The repository could not be found; \
+Check the repository owner and names: '{repos_and_owners_string}")
         print_error_messages(e)
         sys.exit(1)
     except github3.exceptions.ConnectionError as e:

test_issue_metrics.py

@@ -112,10 +112,10 @@ def test_get_per_issue_metrics_with_hide_envs(self):
         ]
 
         # Call the function and check the result
-        with unittest.mock.patch(  # type:ignore
+        with unittest.mock.patch(  # type: ignore
             "issue_metrics.measure_time_to_first_response",
             measure_time_to_first_response,
-        ), unittest.mock.patch(  # type:ignore
+        ), unittest.mock.patch(  # type: ignore
             "issue_metrics.measure_time_to_close", measure_time_to_close
         ):
             (
@@ -221,10 +221,10 @@ def test_get_per_issue_metrics_without_hide_envs(self):
         ]
 
         # Call the function and check the result
-        with unittest.mock.patch(  # type:ignore
+        with unittest.mock.patch(  # type: ignore
             "issue_metrics.measure_time_to_first_response",
             measure_time_to_first_response,
-        ), unittest.mock.patch(  # type:ignore
+        ), unittest.mock.patch(  # type: ignore
             "issue_metrics.measure_time_to_close", measure_time_to_close
         ):
             (
@@ -326,10 +326,10 @@ def test_get_per_issue_metrics_with_ignore_users(self):
         ]
 
         # Call the function and check the result
-        with unittest.mock.patch(  # type:ignore
+        with unittest.mock.patch(  # type: ignore
             "issue_metrics.measure_time_to_first_response",
             measure_time_to_first_response,
-        ), unittest.mock.patch(  # type:ignore
+        ), unittest.mock.patch(  # type: ignore
             "issue_metrics.measure_time_to_close", measure_time_to_close
         ):
             (