review: only look for LGTMs in <p>'s

Fixes: nodejs/node#10657
TimothyGu · Feb 1, 2017 · ca1c671 · ca1c671
1 parent 7cf8cc1
commit ca1c671
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/review.js b/review.js
@@ -351,8 +351,8 @@
       const href = a.getAttribute('href')
       if (!href) continue
       const login = href.slice(1).toLowerCase()
-      const body = comment.querySelector('.comment-body')
-      if (body && LGTM_RE.test(body.innerHTML)) {
+      const paragraphs = comment.querySelectorAll('.comment-body > p')
+      if (Array.from(paragraphs).some(p => LGTM_RE.test(p.innerHTML))) {
         revs.push(login)
       }
     }