diff --git a/Dangerfile b/Dangerfile index 86adae6c..115931df 100644 --- a/Dangerfile +++ b/Dangerfile @@ -1,4 +1,7 @@ -#Check for changes to README.md +# Danger CI configuration file +# https://danger.systems/guides/getting_started.html + +# Check for changes to README.md has_readme_changes = git.modified_files.include?("README.md") # Ensure there is a summary for a pull request @@ -8,7 +11,7 @@ fail 'Please provide a summary in the Pull Request description' if github.pr_bod warn 'Please check PR guidelines and check the boxes.' if github.pr_body.include? '- [ ]' # Warn if pull request is not updated -warn 'Please update the Pull Request title to contain the script name' if github.pr_title.include? 'Update README.md' +warn 'Please provide a descriptive title for the Pull Request' if github.pr_title.include? 'Update README.md' # Warn when there are merge commits in the diff warn 'Please rebase to get rid of the merge commits in this Pull Request' if git.commits.any? { |c| c.message =~ /^Merge branch 'master'/ } diff --git a/Makefile b/Makefile index 4886ccea..d5676eaf 100755 --- a/Makefile +++ b/Makefile @@ -26,7 +26,7 @@ listlicenses: @sed -n -e '/BEGIN SOFTWARE LIST/,/END SOFTWARE LIST/ p' README.md | egrep --only-matching '([Aa0-Zz9]|\s|\.|-)*` `' README.md | sort --unique syntaxerrors: - @echo -e "\n Syntax errors:" + @echo -e "\nSyntax errors:" @! sed -n -e '/BEGIN SOFTWARE LIST/,/END SOFTWARE LIST/ p' README.md | egrep '\)\(|``|\)`' contrib: