Unverified Commit ee82b830 authored by Jason Field's avatar Jason Field Committed by GitHub

Merge pull request #138 from sous-chefs/automated/standardfiles

Automated PR: Standardising Files
parents c4accf97 dfed836e
......@@ -18,7 +18,7 @@ def code_changes?
end
def test_changes?
tests = %w(spec test .kitchen.yml .kitchen.dokken.yml)
tests = %w(spec test kitchen.yml kitchen.dokken.yml)
tests.each do |location|
return true unless git.modified_files.grep(/#{location}/).empty?
end
......@@ -36,6 +36,11 @@ if !git.modified_files.include?('CHANGELOG.md') && code_changes?
failure 'Please include a CHANGELOG entry.'
end
# Require Major Minor Patch version labels
unless github.pr_labels.grep /minor|major|patch/i
warn 'Please add a release label to this pull request'
end
# A sanity check for tests.
if git.lines_of_code > 5 && code_changes? && !test_changes?
warn 'This Pull Request is probably missing tests.'
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment