forked from espressif/esp-idf
Merge branch 'feature/danger_commit_email' into 'master'
ci: danger: check author/committer email Closes IDF-6860 See merge request espressif/esp-idf!22646
This commit is contained in:
@@ -17,6 +17,7 @@ async function runChecks() {
|
|||||||
// Checks for MR commits
|
// Checks for MR commits
|
||||||
require("./mrCommitsTooManyCommits.js")();
|
require("./mrCommitsTooManyCommits.js")();
|
||||||
require("./mrCommitsCommitMessage.js")();
|
require("./mrCommitsCommitMessage.js")();
|
||||||
|
require("./mrCommitsEmail.js")();
|
||||||
|
|
||||||
// Checks for MR code
|
// Checks for MR code
|
||||||
require("./mrSizeTooLarge.js")();
|
require("./mrSizeTooLarge.js")();
|
||||||
|
16
.gitlab/dangerjs/mrCommitsEmail.js
Normal file
16
.gitlab/dangerjs/mrCommitsEmail.js
Normal file
@@ -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.`
|
||||||
|
);
|
||||||
|
}
|
||||||
|
};
|
Reference in New Issue
Block a user