Run black on scripts

Author: sepehr-rs

scripts/check_markup.py

@@ -77,7 +77,11 @@ def check_file(path: Path) -> int:
 
         if findings:
             problems += 1
-            loc = f" ({entry.occurrences[0][0]}:{entry.occurrences[0][1]})" if entry.occurrences else ""
+            loc = (
+                f" ({entry.occurrences[0][0]}:{entry.occurrences[0][1]})"
+                if entry.occurrences
+                else ""
+            )
             tag = " [fuzzy]" if entry.fuzzy else ""
             print(f"{path}{loc}{tag}: {'; '.join(findings)}")
             print(f"    msgid : {entry.msgid[:100]}")

scripts/generate_status_table.py

@@ -28,7 +28,11 @@
 
 def file_stats(path: Path):
     po = polib.pofile(str(path))
-    return len(po.translated_entries()), len(po.fuzzy_entries()), len(po.untranslated_entries())
+    return (
+        len(po.translated_entries()),
+        len(po.fuzzy_entries()),
+        len(po.untranslated_entries()),
+    )
 
 
 def build_table() -> str:
@@ -58,8 +62,11 @@ def build_table() -> str:
 
 def main():
     parser = argparse.ArgumentParser(description=__doc__)
-    parser.add_argument("--file", default="STATUS.md",
-                         help="Markdown file containing the marker block to update")
+    parser.add_argument(
+        "--file",
+        default="STATUS.md",
+        help="Markdown file containing the marker block to update",
+    )
     args = parser.parse_args()
 
     target = REPO_ROOT / args.file

scripts/team_stats.py

@@ -38,15 +38,16 @@
 REPO_ROOT = Path(__file__).resolve().parent.parent
 
 BOT_NAME_RE = re.compile(
-    r'github[^\w]*actions|\[bot\]|not committed yet', re.IGNORECASE)
+    r"github[^\w]*actions|\[bot\]|not committed yet", re.IGNORECASE
+)
 BOT_EMAIL_RE = re.compile(
-    r'github-actions|transifex|\[bot\]|@users\.noreply\.github\.com',
+    r"github-actions|transifex|\[bot\]|@users\.noreply\.github\.com",
     re.IGNORECASE,
 )
 MECHANICAL_SUBJECT_RE = re.compile(
-    r'^(?:sync\s+translations\s+with\s+cpython\b'
-    r'|update\s+\.po\s+files(?:\s*\(\d+\))?\s*$'
-    r'|update\s+farsi\s+translations\s+from\s+transifex\b)',
+    r"^(?:sync\s+translations\s+with\s+cpython\b"
+    r"|update\s+\.po\s+files(?:\s*\(\d+\))?\s*$"
+    r"|update\s+farsi\s+translations\s+from\s+transifex\b)",
     re.IGNORECASE,
 )
 
@@ -70,15 +71,16 @@
 SKIP_DIRS = {".git", ".cpython-src", ".venv", "__pycache__", "venv"}
 
 TEAMMD_ROW_RE = re.compile(
-    r'^\|\s*(?P<user>[^|]+?)\s*\|\s*(?P<role>[^|]+?)\s*\|\s*'
-    r'(?P<t>\d+(?:\s*\([^)]*\))?)\s*\|$'
+    r"^\|\s*(?P<user>[^|]+?)\s*\|\s*(?P<role>[^|]+?)\s*\|\s*"
+    r"(?P<t>\d+(?:\s*\([^)]*\))?)\s*\|$"
 )
 
 
 # ---------------------------------------------------------------------------
 # Privacy helper
 # ---------------------------------------------------------------------------
 
+
 def redact_email(email: str) -> str:
     """Return a partially redacted email for warning messages.
 
@@ -109,6 +111,7 @@ def _blur(s: str) -> str:
 # Git helpers
 # ---------------------------------------------------------------------------
 
+
 def _is_mechanical(subject: str) -> bool:
     return bool(MECHANICAL_SUBJECT_RE.search(subject))
 
@@ -124,9 +127,18 @@ def git_blame_porcelain(path: Path) -> dict[str, dict]:
     ``lines`` (a set of 1-based line numbers blamed to that commit).
     """
     result = subprocess.run(
-        ["git", "-C", str(REPO_ROOT), "blame", "--porcelain",
-         "--", str(path.resolve().relative_to(REPO_ROOT))],
-        capture_output=True, text=True, check=False,
+        [
+            "git",
+            "-C",
+            str(REPO_ROOT),
+            "blame",
+            "--porcelain",
+            "--",
+            str(path.resolve().relative_to(REPO_ROOT)),
+        ],
+        capture_output=True,
+        text=True,
+        check=False,
     )
     if result.returncode != 0:
         return {}
@@ -137,20 +149,26 @@ def git_blame_porcelain(path: Path) -> dict[str, dict]:
     for raw_line in result.stdout.splitlines():
         # Commit header: "<40-char-hash> <orig-line> <result-line> [<num-lines>]"
         parts = raw_line.split()
-        if len(parts) >= 3 and len(parts[0]) == 40 and parts[0].isalnum() and parts[1].isdigit() and parts[2].isdigit():
+        if (
+            len(parts) >= 3
+            and len(parts[0]) == 40
+            and parts[0].isalnum()
+            and parts[1].isdigit()
+            and parts[2].isdigit()
+        ):
             h = parts[0]
             result_line = int(parts[2])
             current_hash = h
             if h not in commits:
                 commits[h] = {"name": "", "email": "", "subject": "", "lines": set()}
             commits[h]["lines"].add(result_line)
         elif raw_line.startswith("author ") and current_hash:
-            commits[current_hash]["name"] = raw_line[len("author "):].strip()
+            commits[current_hash]["name"] = raw_line[len("author ") :].strip()
         elif raw_line.startswith("author-mail ") and current_hash:
-            email = raw_line[len("author-mail "):].strip().strip("<>")
+            email = raw_line[len("author-mail ") :].strip().strip("<>")
             commits[current_hash]["email"] = email.lower()
         elif raw_line.startswith("summary ") and current_hash:
-            commits[current_hash]["subject"] = raw_line[len("summary "):]
+            commits[current_hash]["subject"] = raw_line[len("summary ") :]
 
     return commits
 
@@ -212,6 +230,7 @@ def real_author_for_lines(
 # .po file walking
 # ---------------------------------------------------------------------------
 
+
 def collect_files(paths: list[str]) -> list[Path]:
     files = []
     for arg in paths:
@@ -314,6 +333,7 @@ def teammd_totals() -> dict[str, int]:
 # Output
 # ---------------------------------------------------------------------------
 
+
 def print_report(counts: dict[str, int]) -> None:
     total = sum(counts.values())
     print(f"Total non-fuzzy translated entries attributed: {total}\n")
@@ -371,17 +391,21 @@ def update_teammd(counts: dict[str, int]) -> None:
 # Entry point
 # ---------------------------------------------------------------------------
 
+
 def main() -> None:
     parser = argparse.ArgumentParser(
         description=__doc__,
         formatter_class=argparse.RawDescriptionHelpFormatter,
     )
     parser.add_argument(
-        "paths", nargs="*", default=["."],
+        "paths",
+        nargs="*",
+        default=["."],
         help="Files or directories to scan (default: whole repo)",
     )
     parser.add_argument(
-        "--update-teammd", action="store_true",
+        "--update-teammd",
+        action="store_true",
         help="rewrite TEAM.md from the computed counts",
     )
     args = parser.parse_args()

scripts/translation_status.py

@@ -24,7 +24,11 @@
 
 def file_stats(path: Path):
     po = polib.pofile(str(path))
-    return len(po.translated_entries()), len(po.fuzzy_entries()), len(po.untranslated_entries())
+    return (
+        len(po.translated_entries()),
+        len(po.fuzzy_entries()),
+        len(po.untranslated_entries()),
+    )
 
 
 def collect_files(paths):
@@ -45,12 +49,23 @@ def main():
     parser = argparse.ArgumentParser(
         description=__doc__, formatter_class=argparse.RawDescriptionHelpFormatter
     )
-    parser.add_argument("paths", nargs="*", default=["."],
-                         help="Files or directories to scan (default: whole repo)")
-    parser.add_argument("--sort", choices=["percent", "untranslated", "name"], default="percent",
-                         help="Sort order (default: percent, least-translated first)")
-    parser.add_argument("--only-incomplete", action="store_true",
-                         help="Hide files that are already fully translated")
+    parser.add_argument(
+        "paths",
+        nargs="*",
+        default=["."],
+        help="Files or directories to scan (default: whole repo)",
+    )
+    parser.add_argument(
+        "--sort",
+        choices=["percent", "untranslated", "name"],
+        default="percent",
+        help="Sort order (default: percent, least-translated first)",
+    )
+    parser.add_argument(
+        "--only-incomplete",
+        action="store_true",
+        help="Hide files that are already fully translated",
+    )
     parser.add_argument("--format", choices=["text", "markdown", "csv"], default="text")
     args = parser.parse_args()
 
@@ -94,7 +109,9 @@ def main():
         print("|---|---:|---:|---:|---:|")
         for path, t, fz, u, total, percent in rows:
             print(f"| `{path}` | {t} | {fz} | {u} | {percent:.1f}% |")
-        print(f"| **TOTAL** | **{total_t}** | **{total_fz}** | **{total_u}** | **{total_percent:.1f}%** |")
+        print(
+            f"| **TOTAL** | **{total_t}** | **{total_fz}** | **{total_u}** | **{total_percent:.1f}%** |"
+        )
         return
 
     name_width = max((len(r[0]) for r in rows), default=4)
@@ -104,9 +121,13 @@ def main():
     for path, t, fz, u, total, percent in rows:
         print(f"{path:<{name_width}}  {t:>10}  {fz:>6}  {u:>12}  {percent:>6.1f}%")
     print("-" * len(header))
-    print(f"{'TOTAL':<{name_width}}  {total_t:>10}  {total_fz:>6}  {total_u:>12}  {total_percent:>6.1f}%")
-    print(f"\n{len(rows)} file(s) shown. "
-          f"{sum(1 for r in rows if r[5] < 100)} file(s) not fully translated.")
+    print(
+        f"{'TOTAL':<{name_width}}  {total_t:>10}  {total_fz:>6}  {total_u:>12}  {total_percent:>6.1f}%"
+    )
+    print(
+        f"\n{len(rows)} file(s) shown. "
+        f"{sum(1 for r in rows if r[5] < 100)} file(s) not fully translated."
+    )
 
 
 if __name__ == "__main__":