From 5e2434fdc5881d93aaf4357ea3c2206b0bbdeb06 Mon Sep 17 00:00:00 2001 From: Joshua Nelson Date: Mon, 24 May 2021 16:01:02 -0500 Subject: [PATCH] Don't ping file reviewers if they are the PR author (take 2) `reviewers` is a list, not a string. --- highfive/newpr.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/highfive/newpr.py b/highfive/newpr.py index 386f9cd3..c422b7ef 100644 --- a/highfive/newpr.py +++ b/highfive/newpr.py @@ -352,7 +352,7 @@ def get_to_mention(self, diff, author): mention_list = [] for mention in to_mention: entry = mentions[mention] - if entry["reviewers"] != author: + if author not in entry["reviewers"]: mention_list.append(entry) return mention_list