mas/Dangerfile

35 lines
1.2 KiB
Text
Raw Normal View History

2019-01-15 22:07:50 -07:00
#!/usr/bin/env ruby
#
# Dangerfile
# mas-cli
#
# https://danger.systems/guides/dangerfile.html
#
2018-01-27 18:58:33 -07:00
# Sometimes it's a README fix, or something like that - which isn't relevant for
# including in a project's CHANGELOG for example
2021-04-28 22:38:03 -07:00
has_app_changes = !git.modified_files.grep(/Sources/).empty?
has_test_changes = !git.modified_files.grep(/Tests/).empty?
2018-01-27 21:12:34 -07:00
# if has_app_changes && !has_test_changes
2018-01-27 21:12:34 -07:00
# warn("Tests were not updated", sticky: false)
# end
# Thanks other people!
message(":tada:") if github.pr_author != "phatblat"
2018-01-27 21:12:34 -07:00
# Mainly to encourage writing up some reasoning about the PR, rather than just leaving a title
if github.pr_body.length < 5
2018-01-27 21:44:13 -07:00
fail ":memo: Please provide a summary in the Pull Request description"
2018-01-27 21:12:34 -07:00
end
2018-01-27 18:58:33 -07:00
# Make it more obvious that a PR is a work in progress and shouldn't be merged yet
2018-01-27 21:44:13 -07:00
warn(":construction: PR is classed as Work in Progress") if github.pr_title.include? "[WIP]"
2018-01-27 18:58:33 -07:00
# Warn when there is a big PR
2018-01-27 21:44:13 -07:00
warn(":dizzy_face: Big PR") if git.lines_of_code > 500
2018-01-27 18:58:33 -07:00
2021-05-08 10:23:59 -07:00
# Don't let testing shortcuts get into main by accident
2018-01-27 21:12:34 -07:00
#fail("fdescribe left in tests") if `grep -r fdescribe Tests/ `.length > 1
#fail("fit left in tests") if `grep -r fit Tests/ `.length > 1