Be explicit about the TODO

foursixnine · Feb 2, 2024 · 82937a6 · 82937a6
1 parent 8221031
commit 82937a6
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/openqabot/types/incident.py b/openqabot/types/incident.py
@@ -186,11 +186,13 @@ def filter_failures(self, results):
         ]
 
 
-@staticmethod
+# pylint: disable=fixme
 # TODO:
-# - move to utils
-# - remove almost duplicated code from Approver.is_job_marked_acceptable_for_incident
+#   - move to utils.py or a better place
+#   - remove almost duplicated code from Approver.is_job_marked_acceptable_for_incident
 #   as approver does not seem to operate over incidents
+#   about the TODO see discussion at https://github.com/openSUSE/qem-bot/pull/154#discussion_r1472721681
+@staticmethod
 def has_ignored_comment(job_id: int, inc: int):
     ret = []
     ret = requests.get(OPENQA_URL + "/api/v1/jobs/%s/comments" % job_id).json()