refactor(markdown): use boolean flag for profile linking and sponsor info

Copilot · zkoppert · Copilot · commit e0bc52fe706e · 2025-07-15T06:15:27.000Z
Co-authored-by: zkoppert &lt;6935431+zkoppert@users.noreply.github.com&gt;
diff --git a/markdown.py b/markdown.py
@@ -159,7 +159,8 @@ def get_contributor_table(
     columns = ["Username", "All Time Contribution Count"]
     if start_date and end_date:
         columns += ["New Contributor"]
-    if sponsor_info == "true":
+    # Add Sponsor URL column if sponsor_info is True
+    if sponsor_info:
         columns += ["Sponsor URL"]
     if start_date and end_date:
         columns += [f"Commits between {start_date} and {end_date}"]
@@ -191,8 +192,8 @@ def get_contributor_table(
                 commit_urls += f"{url}, "
         new_contributor = collaborator.new_contributor
 
-        # Determine prefix for username: hyperlink if link_to_profile equals 'true' (string or bool)
-        prefix = "@" if str(link_to_profile).strip().lower() == "true" else ""
+        # Determine prefix for username based on boolean link_to_profile
+        prefix = "@" if link_to_profile else ""
         row = f"| {prefix}{username} | {contribution_count} |"
         if "New Contributor" in columns:
             row += f" {new_contributor} |"
diff --git a/test_markdown.py b/test_markdown.py
@@ -48,7 +48,7 @@ def test_write_to_markdown(self, mock_file):
             "2023-01-02",
             None,
             "org/repo",
-            "false",
+            False,
             True,
             ghe,
         )
@@ -104,7 +104,7 @@ def test_write_to_markdown_with_sponsors(self, mock_file):
             "2023-01-02",
             None,
             "org/repo",
-            "true",
+            True,
             True,
             ghe,
         )
@@ -160,7 +160,7 @@ def test_write_to_markdown_without_link_to_profile(self, mock_file):
             "2023-01-02",
             None,
             "org/repo",
-            "false",
+            False,
             False,
             ghe,
         )
@@ -216,8 +216,8 @@ def test_write_to_markdown_with_string_false_link_to_profile(self, mock_file):
             "2023-01-02",
             None,
             "org/repo",
-            "false",
-            "false",
+            False,
+            False,
             ghe,
         )
 
