pr_checker: show the pr author email and their status

joyeecheung · joyeecheung · commit edd107a70657 · 2017-11-23T23:23:27.000+08:00
diff --git a/lib/pr_checker.js b/lib/pr_checker.js
@@ -268,8 +268,8 @@ class PRChecker {
       return true;
     }
 
-    const prAuthor = pr.author.login;
-    cli.warn(`PR author is: @${prAuthor}`);
+    const prAuthor = `${pr.author.login}(${pr.author.email})`;
+    cli.warn(`PR author is a new contributor: @${prAuthor}`);
     for (const c of oddCommits) {
       const { oid, author } = c.commit;
       const hash = oid.slice(0, 7);
diff --git a/test/unit/pr_checker.test.js b/test/unit/pr_checker.test.js
@@ -530,7 +530,7 @@ describe('PRChecker', () => {
 
       const expectedLogs = {
         warn: [
-          ['PR author is: @pr_author'],
+          ['PR author is a new contributor: @pr_author(pr_author@example.com)'],
           ['- commit e3ad7c7 is authored by test@example.com'],
           ['- commit da39a3e is authored by test@example.com']
         ]

Original file line number	Diff line number	Diff line change
`@@ -530,7 +530,7 @@ describe('PRChecker', () => {`
`530`	`530`
`531`	`531`	`const expectedLogs = {`
`532`	`532`	`warn: [`
`533`		`- ['PR author is: @pr_author'],`
	`533`	`+ ['PR author is a new contributor: @pr_author(pr_author@example.com)'],`
`534`	`534`	`['- commit e3ad7c7 is authored by test@example.com'],`
`535`	`535`	`['- commit da39a3e is authored by test@example.com']`
`536`	`536`	`]`