X-Git-Url: https://git.openstreetmap.org/chef.git/blobdiff_plain/375d9c18ecbcc81fa7f5c4b352a87ab02950a219..4a9eb655d873ad80e9ada63056405c98c7a3e7b0:/hooks/pre-commit diff --git a/hooks/pre-commit b/hooks/pre-commit index 55d23300c..4e45ba7b5 100755 --- a/hooks/pre-commit +++ b/hooks/pre-commit @@ -3,7 +3,7 @@ ok = true if IO.popen(["git", "ls-files", "--unmerged"]).read.empty? - need_stash = IO.popen(%w(git diff)).read.length > 0 + need_stash = !IO.popen(%w(git diff)).read.empty? system("git", "stash", "save", "--keep-index", "--quiet") if need_stash @@ -15,11 +15,11 @@ if IO.popen(["git", "ls-files", "--unmerged"]).read.empty? file =~ /\.rb$/ || `file --brief --mime-type #{file}` == "text/x-ruby\n" end - ok &&= system("rubocop", *ruby_files) unless ruby_files.empty? + ok &&= system("bundle", "exec", "rubocop", *ruby_files) unless ruby_files.empty? cookbooks = files.grep(%r{(cookbooks/[^/]+)/}) { Regexp.last_match(1) }.uniq - ok &&= system("foodcritic", "-f", "any", *cookbooks) unless cookbooks.empty? + ok &&= system("bundle", "exec", "foodcritic", "-f", "any", *cookbooks) unless cookbooks.empty? system("git", "stash", "pop", "--quiet") if need_stash else