Use the same regex as the Actor class when determining co-authors.

itsluketwist · itsluketwist · commit 09f8a1b7b674 · 2022-08-24T19:05:45.000+01:00
diff --git a/git/objects/commit.py b/git/objects/commit.py
@@ -752,11 +752,11 @@ def co_authors(self) -> List[Actor]:
 
         if self.message:
             results = re.findall(
-                r"^Co-authored-by: ((?:\w|\-| ){0,38} <\S*>)$",
+                r"^Co-authored-by: (.*) <(.*?)>$",
                 self.message,
                 re.MULTILINE,
             )
-            for author_string in results:
-                co_authors.append(Actor._from_string(author_string))
+            for author in results:
+                co_authors.append(Actor(*author))
 
         return co_authors
diff --git a/test/test_commit.py b/test/test_commit.py
@@ -517,7 +517,7 @@ def test_commit_co_authors(self):
 Co-authored-by: Test User 1 <602352+test@users.noreply.github.com>
 Co-authored-by: test_user_2 <another_user-email@github.com>
 Co_authored_by: test_user_x <test@github.com>
-Co-authored-by: test_user_y <poorly formatted email @github.com>
+Co-authored-by: test_user_y <test@github.com> text
 Co-authored-by: test_user_3 <test_user_3@github.com>"""
         assert commit.co_authors == [
             Actor("Test User 1", "602352+test@users.noreply.github.com"),

-Original file line number
+Diff line change
 Co-authored-by: Test User 1 <[email protected]>
 Co-authored-by: test_user_2 <[email protected]>
 Co_authored_by: test_user_x <[email protected]>
 -Co-authored-by: test_user_y <poorly formatted email @github.com>
 +Co-authored-by: test_user_y <test@github.com> text
 Co-authored-by: test_user_3 <[email protected]>"""
         assert commit.co_authors == [
             Actor("Test User 1", "[email protected]"),