[ci] use two-level header for more succinct table (hail-is#11447)

* [ci] use two-level header for more succinct table * remove unnecessary newline * fix * fix jinja2 * Update pr-table.html * Update pr-table.html
daniel-goldstein · Mar 3, 2022 · 7657cc7 · 7657cc7
1 parent 9e694a6
commit 7657cc7
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 14 deletions.
diff --git a/ci/ci/ci.py b/ci/ci/ci.py
@@ -97,6 +97,7 @@ class WatchedBranchConfig(TypedDict):
     deploy_state: Optional[str]
     repo: str
     prs: List[PRConfig]
+    gh_status_names: Set[str]
 
 
 async def watched_branch_config(app, wb: WatchedBranch, index: int) -> WatchedBranchConfig:
@@ -105,6 +106,7 @@ async def watched_branch_config(app, wb: WatchedBranch, index: int) -> WatchedBr
     else:
         pr_configs = []
     # FIXME recent deploy history
+    gh_status_names = {k for pr in pr_configs for k in pr['gh_statuses'].keys()}
     return {
         'index': index,
         'branch': wb.branch.short_str(),
@@ -114,6 +116,7 @@ async def watched_branch_config(app, wb: WatchedBranch, index: int) -> WatchedBr
         'deploy_state': wb.deploy_state,
         'repo': wb.branch.repo.short_str(),
         'prs': pr_configs,
+        'gh_status_names': gh_status_names,
     }
 
 

diff --git a/ci/ci/templates/pr-table.html b/ci/ci/templates/pr-table.html
@@ -4,13 +4,18 @@
   <table id="{{ id }}" class="data-table">
   <thead>
     <tr>
-      <th>PR</th>
-      <th>Build State</th>
-      <th>Github Statuses</th>
-      <th>Labels</th>
-      <th>Branch</th>
-      <th>Review State</th>
-      <th>Author</th>
+      <th rowspan="2">PR</th>
+      <th rowspan="2">Build State</th>
+      <th colspan="{{ wb.gh_status_names | length }}">GitHub Statuses</th>
+      <th rowspan="2">Labels</th>
+      <th rowspan="2">Branch</th>
+      <th rowspan="2">Review State</th>
+      <th rowspan="2">Author</th>
+    </tr>
+    <tr>
+      {% for name in wb.gh_status_names %}
+      <th>{{ name }}</th>
+      {% endfor %}
     </tr>
   </thead>
   <tbody>
@@ -31,13 +36,9 @@
           *
         {% endif %}
       </td>
-      <td>
-        <ol>
-          {% for context, status in pr['gh_statuses'].items() %}
-          <li>{{ context }}: {{ status }}</li>
-          {% endfor %}
-        </ol>
-      </td>
+      {% for name in wb.gh_status_names %}
+      <td>{{ pr['gh_statuses'][name] | default("") }}</td>
+      {% endfor %}
       <td>
         {{ pr.labels|join(", ") }}
       </td>