From: Mauro Carvalho Chehab <mchehab@kernel.org>
Date: Wed, 27 May 2026 07:49:00 +0000 (+0000)
Subject: patch.py: better handle color states
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=dbffab9d8b099136b18c40aeda98af5b117fe1c5;p=thirdparty%2Fpatchwork.git

patch.py: better handle color states

The current logic applies checks color only once. That's
weird and makes harder for people to properly idenfify
CI checks when there are multiple CI reports at the same line.

Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
---

diff --git a/patchwork/templatetags/patch.py b/patchwork/templatetags/patch.py
index d3f023f7..c22dfa33 100644
--- a/patchwork/templatetags/patch.py
+++ b/patchwork/templatetags/patch.py
@@ -39,14 +39,9 @@ def patch_checks(patch):
     counts = patch.check_count
 
     check_elements = []
-    use_color = True
     for state in required[::-1]:
         if counts[state]:
-            if use_color:
-                use_color = False
-                color = dict(Check.STATE_CHOICES).get(state)
-            else:
-                color = ''
+            color = dict(Check.STATE_CHOICES).get(state)
             count = str(counts[state])
         else:
             color = ''