Fix json gen and minor improvements

Author: StanFromIreland

completion.py

@@ -3,7 +3,7 @@
 from functools import cache
 from pathlib import Path
 from tempfile import TemporaryDirectory
-from typing import Literal
+from typing import Literal, List
 
 import git
 from potodo import potodo
@@ -36,7 +36,7 @@ def get_completion(
             )
         except git.GitCommandError:
             print(f'failed to clone {repo} {branch}')
-            translators_data = TranslatorsData(0, set(), False)
+            translators_data = TranslatorsData(0, [], False)
             branch = ''
             continue
         else:
@@ -85,5 +85,5 @@ def get_completion(
 @dataclass(frozen=True)
 class TranslatorsData:
     number: int
-    list: set
+    list: List
     link: str | Literal[False]

generate.py

@@ -66,7 +66,7 @@ def get_project_data(
         completion, translators_data, branch, change = get_completion(clones_dir, repo)
     else:
         completion = 0.0
-        translators_data = TranslatorsData(0, set(), False)
+        translators_data = TranslatorsData(0, [], False)
         change = 0.0
         branch = ''
 
@@ -137,6 +137,7 @@ class LanguageProjectData:
     Path('build/logo.png').write_bytes(Path('src/logo.png').read_bytes())
     Path('build/index.html').write_text(index)
     Path('build/chart.html').write_text(chart)
+
     Path('build/index.json').write_text(
-        json.dumps(completion_progress, indent=2, default=asdict)
+        json.dumps([asdict(project) for project in completion_progress], indent=2)
     )

templates/index.html.jinja

@@ -26,4 +26,6 @@
         {% endfor %}
       </div>
     </div>
+
+  <p style="text-align: center;">Last updated at {{ generation_time.strftime('%A, %-d %B %Y, %-H:%M:%S %Z') }} (in {{ duration // 60 }}:{{ "{:02}".format(duration % 60) }} minutes).</p>
 {% endblock %}

templates/language.html.jinja

@@ -17,28 +17,27 @@
     <h6 style="text-align: center;">This project is not built yet.</h6>
   {% endif %}
 
-    <h4 style="text-align: center;">
-      {% if project.translators.link %}
-        <a href="{{ project.translators.link }}" style="color: #1595fe;">Contribute!</a>
-      {% elif project.uses_platform %}
-        <a href="https://app.transifex.com/join/?o=python-doc&p=python-newest&t=opensource" style="color: #1595fe;">Contribute!</a>
-      {% else %}
-        <a href="https://devguide.python.org/documentation/translations/translating/" style="color: #1595fe;">Contribute!</a>
-      {% endif %}
-      •
-      <a href="https://github.com/{{ project.repository }}" style="color: #1595fe;">Repository</a>
-    </h4>
+  <h4 style="text-align: center;">
+    {% if project.translators.link %}
+      <a href="{{ project.translators.link }}" style="color: #1595fe;">Contribute!</a>
+    {% elif project.uses_platform %}
+      <a href="https://app.transifex.com/join/?o=python-doc&p=python-newest&t=opensource" style="color: #1595fe;">Contribute!</a>
+    {% else %}
+      <a href="https://devguide.python.org/documentation/translations/translating/" style="color: #1595fe;">Contribute!</a>
+    {% endif %}
+    •
+    <a href="https://github.com/{{ project.repository }}" style="color: #1595fe;">Repository</a>
+  </h4>
 
-<div class="row">
-  <div class="card-container w-100">
-    <div class="card">
-      <div class="card-body">
-        <h3 class="card-title">Translators ({{ project.translators.number }})</h3>
-        <h4 class="card-subtitle mb-2 text-muted">{{ project.translators.list | join(', ') }}</h4>
+  <div class="row">
+    <div class="card-container w-100">
+      <div class="card">
+        <div class="card-body">
+          <h3 class="card-title">Translators ({{ project.translators.number }})</h3>
+          <h4 class="card-subtitle mb-2 text-muted">{{ project.translators.list | join(', ') }}</h4>
+        </div>
       </div>
     </div>
   </div>
 </div>
-
-</div>
 {% endblock %}

translators.py

@@ -13,12 +13,12 @@ def _format_translator(translator: str) -> str:
     return sub(r'<.*?>', '', translator).strip()
 
 
-def get_number(path: Path) -> tuple[set, int]:
+def get_number(path: Path) -> tuple[list, int]:
     from_po_headers = set(yield_from_headers(path))
     from_git_history = set(get_from_git_history(path))
     from_translators_file = set(get_from_translators_file(path))
     largest_set = max(from_po_headers, from_git_history, from_translators_file)
-    return largest_set, len(largest_set)
+    return list(largest_set), len(largest_set)
 
 
 def get_from_git_history(path: Path) -> list[str]: