Be explicit about the TODO

openSUSE · Mar 5, 2024 · 36ed6db · 36ed6db
1 parent d5b05e2
commit 36ed6db
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/openqabot/types/incident.py b/openqabot/types/incident.py
@@ -187,11 +187,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()