commit
0ec4be74fa
@ -1,23 +1,33 @@
|
|||||||
|
#Check for changes to README.md
|
||||||
|
has_readme_changes = git.modified_files.include?("README.md")
|
||||||
|
|
||||||
# Ensure there is a summary for a pull request
|
# 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
|
fail 'Please provide a summary in the Pull Request description' if github.pr_body.length < 5
|
||||||
|
|
||||||
|
# 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 when there are merge commits in the diff
|
# 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'/ }
|
warn 'Please rebase to get rid of the merge commits in this Pull Request' if git.commits.any? { |c| c.message =~ /^Merge branch 'master'/ }
|
||||||
|
|
||||||
# Check links
|
# Check links
|
||||||
require 'json'
|
if has_readme_changes
|
||||||
results = File.read 'ab-results-README.md-markdown-table.json'
|
require 'json'
|
||||||
j = JSON.parse results
|
results = File.read 'ab-results-README.md-markdown-table.json'
|
||||||
if j['error']==true
|
j = JSON.parse results
|
||||||
|
if j['error']==true
|
||||||
fail j['title']
|
fail j['title']
|
||||||
markdown j['message']
|
markdown j['message']
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
# Check syntx
|
# Check syntax
|
||||||
require 'json'
|
if has_readme_changes
|
||||||
results = File.read 'syntaxcheck.json'
|
require 'json'
|
||||||
j = JSON.parse results
|
results = File.read 'syntaxcheck.json'
|
||||||
if j['error']==true
|
j = JSON.parse results
|
||||||
|
if j['error']==true
|
||||||
fail j['title']
|
fail j['title']
|
||||||
markdown j['message']
|
markdown j['message']
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
Loading…
Reference in new issue