[2/4] scripts/machine-summary: don't mark denied patches as good

Message ID 20220125165936.1456059-2-ross.burton@arm.com
State New
Headers show
Series [1/4] scripts/machine-summary: remove obsolete is_old test | expand

Commit Message

Ross Burton Jan. 25, 2022, 4:59 p.m. UTC
A patch in denied state isn't good, it's bad.

Signed-off-by: Ross Burton <ross.burton@arm.com>
---
 scripts/report-details.html.jinja | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Patch

diff --git a/scripts/report-details.html.jinja b/scripts/report-details.html.jinja
index ba66fcf0..19ede510 100644
--- a/scripts/report-details.html.jinja
+++ b/scripts/report-details.html.jinja
@@ -6,9 +6,9 @@ 
   {% set status = status.split()[0] %}
   {% if status in ("Unknown", "Pending") %}
     {% set class = "is-danger" %}
-  {% elif status in ("Backport", "Accepted", "Inappropriate") %}
+  {% elif status in ("Backport", "Accepted", "Inappropriate", "Denied") %}
     {% set class = "is-success" %}
-  {% elif status in ("Submitted", "Denied") %}
+  {% elif status in ("Submitted",) %}
     {% set class = "is-info" %}
   {% else %}
     {% set class = "is-info" %}