Merging workflow

Author: m-aciek

completion.py

@@ -2,6 +2,7 @@
 from functools import cache
 from pathlib import Path
 from tempfile import TemporaryDirectory
+from typing import Literal
 
 import git
 from potodo import potodo
@@ -18,7 +19,9 @@ def branches_from_devguide(devguide_dir: Path) -> list[str]:
     ]
 
 
-def get_completion(clones_dir: str, repo: str) -> tuple[float, int]:
+def get_completion(
+    clones_dir: str, repo: str
+) -> tuple[float, int, str | Literal[False]]:
     clone_path = Path(clones_dir, repo)
     for branch in branches_from_devguide(Path(clones_dir, 'devguide')) + ['master']:
         try:
@@ -28,9 +31,11 @@ def get_completion(clones_dir: str, repo: str) -> tuple[float, int]:
         except git.GitCommandError:
             print(f'failed to clone {repo} {branch}')
             translators_number = 0
+            translators_link: str | Literal[False] = False
             continue
         else:
             translators_number = translators.get_number(clone_path)
+            translators_link = translators.get_link(clone_path, repo, branch)
             break
     with TemporaryDirectory() as tmpdir:
         completion = potodo.merge_and_scan_path(
@@ -40,4 +45,4 @@ def get_completion(clones_dir: str, repo: str) -> tuple[float, int]:
             hide_reserved=False,
             api_url='',
         ).completion
-    return completion, translators_number
+    return completion, translators_number, translators_link

generate.py

@@ -29,7 +29,18 @@
 
 def get_completion_progress() -> (
     Iterator[
-        tuple[str, str, float, int, int, bool, bool | None, bool, str | Literal[False]]
+        tuple[
+            str,
+            str,
+            float,
+            int,
+            str | Literal[False],
+            int,
+            bool,
+            bool | None,
+            bool,
+            str | Literal[False],
+        ]
     ]
 ):
     with TemporaryDirectory() as clones_dir:
@@ -58,15 +69,27 @@ def get_completion_progress() -> (
             tx = lang in contribute.pulling_from_transifex
             contrib_link = contribute.get_contrib_link(lang)
             if not repo:
-                yield lang, cast(str, repo), 0.0, 0, 0, built, in_switcher, False, False
+                yield (
+                    lang,
+                    cast(str, repo),
+                    0.0,
+                    0,
+                    False,
+                    0,
+                    built,
+                    in_switcher,
+                    False,
+                    False,
+                )
                 continue
-            completion, translators = get_completion(clones_dir, repo)
+            completion, translators, translators_link = get_completion(clones_dir, repo)
             visitors_num = visitors.get_number_of_visitors(lang) if built else 0
             yield (
                 lang,
                 repo,
                 completion,
                 translators,
+                translators_link,
                 visitors_num,
                 built,
                 in_switcher,

template.html.jinja

@@ -17,7 +17,7 @@
 </tr>
 </thead>
 <tbody>
-{% for language, repo, completion, translators, visitors, build, in_switcher, on_platform, contrib_link in completion_progress | sort(attribute='2,3') | reverse %}
+{% for language, repo, completion, translators, translators_link, visitors, build, in_switcher, on_platform, contrib_link in completion_progress | sort(attribute='2,3') | reverse %}
 <tr>
   {% if repo %}
   <td data-label="language">
@@ -46,10 +46,14 @@
         {{ '{:,}'.format(visitors) }}
       </a>
     {% else %}
-      0
+      {{ '{:,}'.format(visitors) }}
     {% endif %}
   </td>
-  <td data-label="translators">{{ translators }}</td>
+  <td data-label="translators">
+    {% if translators_link %}<a href="{{ translators_link }}" target="_blank">{% endif %}
+    {{ translators }}
+    {% if translators_link %}</a>{% endif %}
+  </td>
   <td data-label="completion">
     <div class="progress-bar" style="width: {{ completion | round(2) }}%;">{{ completion | round(2) }}%</div>
   </td>

translators.py

@@ -1,5 +1,6 @@
 from collections.abc import Iterator
 from pathlib import Path
+from typing import Literal
 
 from git import Repo
 from polib import pofile
@@ -30,3 +31,10 @@ def yield_from_headers(path: Path) -> Iterator[str]:
                 yield translator_record
             else:
                 yield translator
+
+
+def get_link(clone_path: Path, repo: str, branch: str) -> str | Literal[False]:
+    return (
+        clone_path.joinpath('TRANSLATORS').exists()
+        and f'https://github.com/{repo}/blob/{branch}/TRANSLATORS'
+    )