Merge branch 'ci/dangerjs_success_log' into 'master'

dangerjs: add success log

See merge request espressif/esp-idf!22470
pull/10856/head
Fu Hanxi 2023-02-23 15:53:11 +08:00
commit 0d97eeb7a5
2 zmienionych plików z 14 dodań i 2 usunięć

Wyświetl plik

@ -1,4 +1,4 @@
import { danger, warn, message } from "danger"
import { danger, warn, message, results } from "danger"
/**
* Check if MR Title contains prefix "Draft: ... or "WIP: ...".
@ -62,10 +62,15 @@ function checkMrJiraLinks() {
const mrCommitMessages = danger.gitlab.commits.map(commit => commit.message);
const matchBlockRelated = mrDescription.match(/\#\# Related.*$/s); // Match MR description starting with line ## Related till the end of MR description
const noRelatedIssues = /No related issues/.test(matchBlockRelated ? matchBlockRelated[0] : ''); // Check if there is "No related issues"
const testJiraLabels = /[A-Z]+-[0-9]+/.test(matchBlockRelated ? matchBlockRelated[0] : ''); // Test if pattern of Jira label "JIRA-1234" or "RDT-311" is in section Related
const ghIssueTicket = /IDFGH-[0-9]+/.test(matchBlockRelated ? matchBlockRelated[0] : ''); // Check if there is JIRA link starts with "IDFGH-*" in MR description, section "Related"
const testGithubLink = /Closes https:\/\/github\.com\/espressif\/esp-idf\/issues\/[0-9]+/
if (mrDescription.toUpperCase().includes("## RELATED") && noRelatedIssues) {
return
}
if (!mrDescription.toUpperCase().includes("## RELATED") || !testJiraLabels) { // Missing section "Related" or missing links to JIRA tickets
return warn("Please add links to JIRA issues to the MR description section `Related`.");
@ -177,3 +182,10 @@ function addRetryLink() {
return markdown(`***\n#### :repeat: If you want to run these checks again, please retry this [DangerJS job](${retryLink})\n***`);
}
addRetryLink();
function printSuccessLog() {
if (results.fails.length === 0 && results.warnings.length === 0 && results.messages.length === 0) {
return message('Good Job! All checks are passing!')
}
}
printSuccessLog();

Wyświetl plik

@ -7,7 +7,7 @@
<!-- Add description of the change here --><!-- Mandatory -->
## Related <!-- Optional -->
<!-- Related Jira issues and Github issues -->
<!-- Related Jira issues and Github issues or write "No related issues"-->
## Release notes <!-- Mandatory -->
<!-- Either state release notes or write "No release notes" -->