X-Git-Url: https://git.openstreetmap.org./chef.git/blobdiff_plain/90a349e0bc558441e65156eabb585c578d93fbfd..3d7c0e60abe0808fb82da2009e8f92963b992369:/hooks/pre-commit?ds=sidebyside diff --git a/hooks/pre-commit b/hooks/pre-commit index cb7bc82d5..449a8c6f2 100755 --- a/hooks/pre-commit +++ b/hooks/pre-commit @@ -1,4 +1,4 @@ -#!/usr/bin/ruby +#!/usr/bin/env ruby ok = true @@ -12,14 +12,10 @@ if IO.popen(["git", "ls-files", "--unmerged"]).read.empty? end.compact ruby_files = files.select do |file| - file =~ /\.rb$/ || (file !~ /\.erb$/ && `file --brief --mime-type #{file}` == "text/x-ruby\n") + file =~ /\.rb$/ || (file !~ /\.erb$/ && %x(file --brief --mime-type #{file}) == "text/x-ruby\n") end - ok &&= system("bundle", "exec", "rubocop", *ruby_files) unless ruby_files.empty? - - cookbooks = files.grep(%r{(cookbooks/[^/]+)/}) { Regexp.last_match(1) }.uniq - - ok &&= system("bundle", "exec", "foodcritic", *cookbooks) unless cookbooks.empty? + ok &&= system("bundle", "exec", "cookstyle", *ruby_files) unless ruby_files.empty? system("git", "stash", "pop", "--quiet") if need_stash else