Also look for Differential Revision: to ensure that this repo uses Ph…

…abricator Signed-off-by: Vihang Mehta <vihang@pixielabs.ai>
ossf · May 4, 2022 · 366844a · 366844a
1 parent 1e2116b
commit 366844a
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/checks/evaluation/code_review.go b/checks/evaluation/code_review.go
@@ -203,7 +203,8 @@ func isReviewedOnPhabricator(c *checker.DefaultBranchCommit, dl checker.DetailLo
 	}
 
 	m := c.CommitMessage
-	if strings.Contains(m, "\nReviewed By: ") {
+	if strings.Contains(m, "\nDifferential Revision: ") &&
+		strings.Contains(m, "\nReviewed By: ") {
 		dl.Debug(&checker.LogMessage{
 			Text: fmt.Sprintf("commit %s was approved through %s", c.SHA, reviewPlatformPhabricator),
 		})