vbatts/cri-o
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #70 from runcom/gometalinter

Browse source 
Makefile: suggests install.tools
This commit is contained in:
Antonio Murdaca 2016-09-28 21:02:31 +02:00 committed by GitHub
commit 7e2460ad73
2 changed files with 1 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
.tool/lint
View file

@ -4,11 +4,6 @@ set -o errexit
set -o nounset set -o nounset
set -o pipefail set -o pipefail
if [ ! $(command -v gometalinter) ]; then
go get -u github.com/alecthomas/gometalinter
gometalinter --update --install
fi
for d in $(find . -type d -not -iwholename '*.git*' -a -not -iname '.tool' -a -not -iwholename '*vendor*'); do for d in $(find . -type d -not -iwholename '*.git*' -a -not -iname '.tool' -a -not -iwholename '*vendor*'); do
gometalinter \ gometalinter \
--exclude='error return value not checked.*(Close|Log|Print).*\(errcheck\)$' \ --exclude='error return value not checked.*(Close|Log|Print).*\(errcheck\)$' \

1
Makefile
View file

@ -22,6 +22,7 @@ help:
@echo " * 'lint' - Execute the source code linter" @echo " * 'lint' - Execute the source code linter"
lint: ${OCID_LINK} lint: ${OCID_LINK}
@which gometalinter > /dev/null 2>/dev/null || (echo "ERROR: gometalinter not found. Consider 'make install.tools' target" && false)
@echo "checking lint" @echo "checking lint"
@./.tool/lint @./.tool/lint