pr_checker: show the pr author email and their status

joyeecheung · joyeecheung · commit 2f5773e88e7e · 2017-11-23T23:13:03.000+08:00
diff --git a/lib/pr_checker.js b/lib/pr_checker.js
@@ -247,8 +247,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
@@ -416,7 +416,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
`@@ -416,7 +416,7 @@ describe('PRChecker', () => {`
`416`	`416`
`417`	`417`	`const expectedLogs = {`
`418`	`418`	`warn: [`
`419`		`- ['PR author is: @pr_author'],`
	`419`	`+ ['PR author is a new contributor: @pr_author(pr_author@example.com)'],`
`420`	`420`	`['- commit e3ad7c7 is authored by test@example.com'],`
`421`	`421`	`['- commit da39a3e is authored by test@example.com']`
`422`	`422`	`]`