Clean up some logic in audit_modified_casks script

This commit is contained in:
Josh Hagins
2015-07-24 13:07:08 -04:00
parent 875c624e59
commit 533e00a345
+9 -8
View File
@@ -69,13 +69,18 @@ audit_cask () {
needs_verification () {
local cask_file="$1"
! is_sha256_no_check "$cask_file" &&
file_diff "$cask_file" | is_relevant_stanza_modified
is_sha256_checked "$cask_file" &&
is_relevant_stanza_modified "$cask_file"
}
is_sha256_no_check () {
is_sha256_checked () {
local cask_file="$1"
grep -q 'sha256\s*:no_check' "$cask_file"
! grep -q 'sha256\s*:no_check' "$cask_file"
}
is_relevant_stanza_modified () {
local cask_file="$1"
file_diff "$cask_file" | grep -Eq '^\+\s*(url|sha256|version)'
}
file_diff () {
@@ -83,10 +88,6 @@ file_diff () {
git diff "$commit_range" -- "$cask_file"
}
is_relevant_stanza_modified () {
grep -Eq '^\+\s*(url|sha256|version)'
}
modified_casks () {
modified_files | grep "^$cask_dir/"
}