From 972b601c36ba8bb37a5d980535f2cbd3131c9e1a Mon Sep 17 00:00:00 2001 From: "Anna Yeddi (Kanterova)" <40773610+anna-yeddi@users.noreply.github.com> Date: Tue, 23 Jan 2024 10:21:51 -0500 Subject: [PATCH] Update severity_inconsistency.py The Bugbot sends an NI to an assignee alone, instead of the triage owner, when the accessibility severity is higher than the severity: > --- > The severity field for this bug is set to S3. However, the accessibility severity is higher, . :ayeddi, could you consider increasing the severity? Another approach could be to remove the "assigned_to" field at all since all the bugs have always "triage_owner" filled in. --- bugbot/rules/severity_inconsistency.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bugbot/rules/severity_inconsistency.py b/bugbot/rules/severity_inconsistency.py index 11313c3e7..75e837615 100644 --- a/bugbot/rules/severity_inconsistency.py +++ b/bugbot/rules/severity_inconsistency.py @@ -30,7 +30,7 @@ def columns(self): return ["id", "summary", "severity", "accessibility_severity"] def get_mail_to_auto_ni(self, bug): - for field in ["assigned_to", "triage_owner"]: + for field in ["triage_owner", "assigned_to"]: person = bug.get(field, "") if person and not utils.is_no_assignee(person): return {"mail": person, "nickname": bug[f"{field}_detail"]["nick"]}