Merge pull request #15 from github/ttfr-is-none

fix: time to first response being none caused sort failure
github · Jun 9, 2023 · 4354219 · 4354219
2 parents ff1064d + e3c04d1
commit 4354219
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/issue_metrics.py b/issue_metrics.py
@@ -187,7 +187,9 @@ def write_to_markdown(
             file.write("no issues found for the given search criteria\n\n")
     else:
         # Sort the issues by time to first response
-        issues_with_metrics.sort(key=lambda x: x.time_to_first_response)
+        issues_with_metrics.sort(
+            key=lambda x: x.time_to_first_response or timedelta.max
+        )
         with file or open("issue_metrics.md", "w", encoding="utf-8") as file:
             file.write("# Issue Metrics\n\n")
             file.write("| Metric | Value |\n")