Merge branch 'feature/danger_commit_email' into 'master'

ci: danger: check author/committer email Closes IDF-6860 See merge request espressif/esp-idf!22646
espressif · Mar 31, 2023 · cac58c5 · cac58c5
2 parents 7c8e855 + eb3b2b5
commit cac58c5
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/.gitlab/dangerjs/dangerfile.js b/.gitlab/dangerjs/dangerfile.js
@@ -17,6 +17,7 @@ async function runChecks() {
     // Checks for MR commits
     require("./mrCommitsTooManyCommits.js")();
     require("./mrCommitsCommitMessage.js")();
+    require("./mrCommitsEmail.js")();
 
     // Checks for MR code
     require("./mrSizeTooLarge.js")();

diff --git a/.gitlab/dangerjs/mrCommitsEmail.js b/.gitlab/dangerjs/mrCommitsEmail.js
@@ -0,0 +1,16 @@
+/**
+ * Check if the author is accidentally making a commit using a personal email
+ *
+ * @dangerjs INFO
+ */
+module.exports = function () {
+    const mrCommitAuthorEmails = danger.gitlab.commits.map(commit => commit.author_email);
+    const mrCommitCommitterEmails = danger.gitlab.commits.map(commit => commit.committer_email);
+	const emailPattern = /.*@espressif\.com/;
+	const filteredEmails = [...mrCommitAuthorEmails, ...mrCommitCommitterEmails].filter((email) => !emailPattern.test(email));
+	if (filteredEmails.length) {
+		return message(
+			`Some of the commits were authored or committed by developers outside Espressif: ${filteredEmails.join(', ')}. Please check if this is expected.`
+		);
+	}
+};