Only look for LGTMs in <p>'s

TimothyGu · TimothyGu · commit b834000c8df5 · 2017-01-29T14:57:06.000-08:00
Fixes: nodejs/node#10657
diff --git a/review.js b/review.js
@@ -349,8 +349,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)
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -349,8 +349,8 @@`
`349`	`349`	`const href = a.getAttribute('href')`
`350`	`350`	`if (!href) continue`
`351`	`351`	`const login = href.slice(1).toLowerCase()`
`352`		`- const body = comment.querySelector('.comment-body')`
`353`		`- if (body && LGTM_RE.test(body.innerHTML)) {`
	`352`	`+ const paragraphs = comment.querySelectorAll('.comment-body > p')`
	`353`	`+ if (Array.from(paragraphs).some(p => LGTM_RE.test(p.innerHTML))) {`
`354`	`354`	`revs.push(login)`
`355`	`355`	`}`
`356`	`356`	`}`