Browse Source

Danger warn instead of fail for checkboxes

pull/880/merge
nrew225 8 years ago
parent
commit
a0524a64ce
  1. 4
      Dangerfile

4
Dangerfile

@ -4,8 +4,8 @@ has_readme_changes = git.modified_files.include?("README.md")
# Ensure there is a summary for a pull request
fail 'Please provide a summary in the Pull Request description' if github.pr_body.length < 5
# Check if PR guidelines are checked.
fail 'Please check PR guidelines and check the boxes.' if github.pr_body.include? '- [ ]'
# Warn if PR guideline boxes are not checked.
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'

Loading…
Cancel
Save