From a303b4f879acae2cb88a899b8d8d33c9bd89bebc Mon Sep 17 00:00:00 2001
From: Rob Mohr <mohrr@google.com>
Date: Thu, 18 Feb 2021 07:49:40 -0800
Subject: [PATCH] pw_presubmit: Better formatting for long times

Change-Id: I3f5ed338718ede5a93900217bd7b4fbe31159c26
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/33120
Pigweed-Auto-Submit: Rob Mohr <mohrr@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed.google.com.iam.gserviceaccount.com>
Reviewed-by: Keir Mierle <keir@google.com>
---
 pw_presubmit/py/pw_presubmit/presubmit.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/pw_presubmit/py/pw_presubmit/presubmit.py b/pw_presubmit/py/pw_presubmit/presubmit.py
index a572a9a39..5ddddf553 100644
--- a/pw_presubmit/py/pw_presubmit/presubmit.py
+++ b/pw_presubmit/py/pw_presubmit/presubmit.py
@@ -83,7 +83,10 @@ def _title(msg, style=_SUMMARY_BOX) -> str:
 
 def _format_time(time_s: float) -> str:
     minutes, seconds = divmod(time_s, 60)
-    return f' {int(minutes)}:{seconds:04.1f}'
+    if minutes < 60:
+        return f' {int(minutes)}:{seconds:04.1f}'
+    hours, minutes = divmod(minutes, 60)
+    return f'{int(hours):d}:{int(minutes):02}:{int(seconds):02}'
 
 
 def _box(style, left, middle, right, box=tools.make_box('><>')) -> str: