Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

Commit

Permalink
Update Makefile
Browse files Browse the repository at this point in the history
  • Loading branch information
@honnix
honnix authored Dec 9, 2019
1 parent 101f258 commit b9bce6d
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions boilerplate/lyft/golang_test_targets/Makefile
View file
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,11 @@ DEP_SHA=1f7c19e5f52f49ffb9f956f64c010be14683468b

.PHONY: lint
lint: #lints the package for common code smells
which golangci-lint || go install github.com/golangci/golangci-lint/cmd/golangci-lint
# install outside of current module, workaround https://github.com/golang/go/issues/30515
which golangci-lint || (cd /tmp && go get github.com/golangci/golangci-lint/cmd/golangci-lint && cd -)
# Calling lint twice here is a hack. The first call seem to fail when internally calling `go list...`
# However, that call seem to have some effects (e.g. https://github.com/golang/go/issues/29452) which, for some
# reason, allows the subsequent calls to succeed.
# However, that call seem to have some effects (e.g. https://github.com/golang/go/issues/29452) which, for some
# reason, allows the subsequent calls to succeed.
GL_DEBUG=linters_output,env golangci-lint run --deadline=5m --exclude deprecated -v

# If code is failing goimports linter, this will fix.
Expand Down

0 comments on commit b9bce6d

Please sign in to comment.