Contents

# Make sure non-trivial amounts of code changes come with corresponding tests
has_app_changes = !git.modified_files.grep(/lib/).empty?
has_spec_changes = !git.modified_files.grep(/spec/).empty?

if git.lines_of_code > 50 && has_app_changes && !has_spec_changes
  warn('There are code changes, but no corresponding tests. '\
         'Please include tests if this PR introduces any modifications in '\
         'behavior.',
       sticky: false)
end

# Mainly to encourage writing up some reasoning about the PR, rather than
# just leaving a title
warn('Please add a detailed summary in the description.') if github.pr_body.length < 5

# Ensure a clean commits history
if git.commits.any? { |c| c.message =~ /^Merge branch '#{github.branch_for_base}'/ }
  warn 'Please rebase to get rid of the merge commits in this PR'
end

# Warn when there is a big PR
warn('This PR is too big! Consider breaking it down into smaller PRs.') if git.lines_of_code > 500

Version data entries

14 entries across 14 versions & 1 rubygems

Version	Path
pushmi_pullyu-1.0.6	Dangerfile
pushmi_pullyu-1.0.5	Dangerfile
pushmi_pullyu-1.0.4	Dangerfile
pushmi_pullyu-1.0.3	Dangerfile
pushmi_pullyu-1.0.2	Dangerfile
pushmi_pullyu-1.0.1	Dangerfile
pushmi_pullyu-0.2.7	Dangerfile
pushmi_pullyu-0.2.6	Dangerfile
pushmi_pullyu-0.2.5	Dangerfile
pushmi_pullyu-0.2.4	Dangerfile
pushmi_pullyu-0.2.3	Dangerfile
pushmi_pullyu-0.2.2	Dangerfile
pushmi_pullyu-0.2.1	Dangerfile
pushmi_pullyu-0.2.0	Dangerfile