This is an automated email from the ASF dual-hosted git repository.

sbp pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tooling-trusted-release.git


The following commit(s) were added to refs/heads/main by this push:
     new 55cd825  Avoid showing a details element when there are no check 
results
55cd825 is described below

commit 55cd8250a8173f6b999eb9968a5bb7e522b67cab
Author: Sean B. Palmer <s...@miscoranda.com>
AuthorDate: Thu Apr 3 15:24:46 2025 +0100

    Avoid showing a details element when there are no check results
---
 atr/templates/draft-review-path.html | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/atr/templates/draft-review-path.html 
b/atr/templates/draft-review-path.html
index 87cc96b..57a52f7 100644
--- a/atr/templates/draft-review-path.html
+++ b/atr/templates/draft-review-path.html
@@ -111,14 +111,14 @@
 
             {% if check_result.message %}<div class="mt-2">{{ 
check_result.message }}</div>{% endif %}
 
-            {% if check_result.data is mapping %}
+            {% if (check_result.data is mapping) and 
(check_result.data.keys()|length > 0) %}
               <details class="mt-2 p-2 bg-light rounded">
                 {% if check_result.status.value in ["failure", "exception"] %}
-                  <summary class="atr-cursor-pointer user-select-none 
p-2">View details ({{ check_result.status.value|title }})</summary>
+                  <summary class="atr-cursor-pointer user-select-none 
p-2">View details</summary>
                 {% elif check_result.status.value == "warning" %}
-                  <summary class="atr-cursor-pointer user-select-none 
p-2">View details (Warning)</summary>
+                  <summary class="atr-cursor-pointer user-select-none 
p-2">View details</summary>
                 {% else %}
-                  <summary class="atr-cursor-pointer user-select-none 
p-2">View detailed results</summary>
+                  <summary class="atr-cursor-pointer user-select-none 
p-2">View details</summary>
                 {% endif %}
 
                 {% if check_result.checker.endswith("rat.check_licenses") %}


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@tooling.apache.org
For additional commands, e-mail: commits-h...@tooling.apache.org

Reply via email to