Merge pull request #26 from weaveworks/this-time-for-sure

Properly exclude vendor from lint
This commit is contained in:
Jonathan Lange
2016-07-15 11:34:39 +01:00
committed by GitHub
2 changed files with 2 additions and 4 deletions

View File

@@ -4,10 +4,8 @@
#
# e.g.
# $ files-with-type text/x-shellscript k8s infra
#
# Assumes `find`, `xargs`, and `file` are all installed.
mime_type=$1
shift
git ls-files | grep -vE '^vendor/' | xargs file --mime-type | grep "${mime_type}" | sed -e 's/:.*$//'
git ls-files "$@" | grep -vE '^vendor/' | xargs file --mime-type | grep "${mime_type}" | sed -e 's/:.*$//'

2
lint
View File

@@ -166,7 +166,7 @@ function lint_files {
function list_files {
if [ $# -gt 0 ]; then
git ls-files --exclude-standard | grep -v '^vendor/'
git ls-files --exclude-standard | grep -vE '(^|/)vendor/'
else
git diff --cached --name-only
fi