Remove default values from acknowledge_coauthors parameters

Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>

Author: Copilot

contributors.py

@@ -112,7 +112,7 @@ def get_all_contributors(
     end_date: str,
     github_connection: object,
     ghe: str,
-    acknowledge_coauthors: bool = False,
+    acknowledge_coauthors: bool,
 ):
     """
     Get all contributors from the organization or repository
@@ -189,7 +189,7 @@ def get_contributors(
     start_date: str,
     end_date: str,
     ghe: str,
-    acknowledge_coauthors: bool = False,
+    acknowledge_coauthors: bool,
 ):
     """
     Get contributors from a single repository and filter by start end dates if present.

test_contributors.py

@@ -30,7 +30,7 @@ def test_get_contributors(self, mock_contributor_stats):
         mock_repo.contributors.return_value = [mock_user]
         mock_repo.full_name = "owner/repo"
 
-        get_contributors(mock_repo, "2022-01-01", "2022-12-31", "")
+        get_contributors(mock_repo, "2022-01-01", "2022-12-31", "", False)
 
         mock_contributor_stats.assert_called_once_with(
             "user",
@@ -64,7 +64,7 @@ def test_get_all_contributors_with_organization(self, mock_get_contributors):
         ghe = ""
 
         result = get_all_contributors(
-            "org", "", "2022-01-01", "2022-12-31", mock_github_connection, ghe
+            "org", "", "2022-01-01", "2022-12-31", mock_github_connection, ghe, False
         )
 
         self.assertEqual(
@@ -107,7 +107,13 @@ def test_get_all_contributors_with_repository(self, mock_get_contributors):
         ghe = ""
 
         result = get_all_contributors(
-            "", ["owner/repo"], "2022-01-01", "2022-12-31", mock_github_connection, ghe
+            "",
+            ["owner/repo"],
+            "2022-01-01",
+            "2022-12-31",
+            mock_github_connection,
+            ghe,
+            False,
         )
 
         self.assertEqual(
@@ -147,7 +153,7 @@ def test_get_contributors_skip_users_with_no_commits(self, mock_contributor_stat
         mock_repo.get_commits.side_effect = StopIteration
         ghe = ""
 
-        get_contributors(mock_repo, "2022-01-01", "2022-12-31", ghe)
+        get_contributors(mock_repo, "2022-01-01", "2022-12-31", ghe, False)
 
         # Note that only user is returned and user2 is not returned here because there were no commits in the date range
         mock_contributor_stats.assert_called_once_with(
@@ -175,7 +181,7 @@ def test_get_contributors_skip_bot(self, mock_contributor_stats):
         mock_repo.get_commits.side_effect = StopIteration
         ghe = ""
 
-        get_contributors(mock_repo, "2022-01-01", "2022-12-31", ghe)
+        get_contributors(mock_repo, "2022-01-01", "2022-12-31", ghe, False)
 
         # Note that only user is returned and user2 is not returned here because there were no commits in the date range
         mock_contributor_stats.isEmpty()
@@ -196,7 +202,7 @@ def test_get_contributors_no_commit_end_date(self, mock_contributor_stats):
         mock_repo.get_commits.side_effect = StopIteration
         ghe = ""
 
-        get_contributors(mock_repo, "2022-01-01", "", ghe)
+        get_contributors(mock_repo, "2022-01-01", "", ghe, False)
 
         # Note that only user is returned and user2 is not returned here because there were no commits in the date range
         mock_contributor_stats.assert_called_once_with(