refactor: add _COUNT_THOUSANDS/_COUNT_MILLIONS constants to match _SECONDS_PER_MINUTE pattern

format_count used bare 1_000 and 1_000_000 literals six times while the
adjacent format_duration already named its thresholds. Replacing the magic
numbers with named constants makes the two formatters consistent and removes
the repeated literals.

Co-authored-by: Ralphify <noreply@ralphify.co>

Author: 2 people

src/ralphify/_output.py

@@ -98,17 +98,19 @@ def format_count(n: int) -> str:
     e.g. 999_950 → ``"1.0M"`` instead of ``"1000.0k"`` (same guard
     as :func:`format_duration`'s 59.95 → ``"1m 0s"``).
     """
-    if n >= 1_000_000:
-        return f"{n / 1_000_000:.1f}M"
-    if n >= 1_000:
+    if n >= _COUNT_MILLIONS:
+        return f"{n / _COUNT_MILLIONS:.1f}M"
+    if n >= _COUNT_THOUSANDS:
         # Use rounded value to avoid "1000.0k" when rounding crosses
         # into the next unit (same guard as format_duration's 59.95→1m).
-        if round(n / 1_000, 1) >= 1_000:
-            return f"{n / 1_000_000:.1f}M"
-        return f"{n / 1_000:.1f}k"
+        if round(n / _COUNT_THOUSANDS, 1) >= _COUNT_THOUSANDS:
+            return f"{n / _COUNT_MILLIONS:.1f}M"
+        return f"{n / _COUNT_THOUSANDS:.1f}k"
     return str(n)
 
 
+_COUNT_THOUSANDS = 1_000
+_COUNT_MILLIONS = 1_000_000
 _SECONDS_PER_MINUTE = 60
 _MINUTES_PER_HOUR = 60