@rianhughes
Please have a look at Makefile

Hi @rianhughes
I tried to implement golangci-lint.yml file
I am facing few problems
make lint gives me this error

golangci-lint not installed. Run 'make install-golangci-lint' to install it.
make: *** [Makefile:26: lint] Error 1

Even though I have installed golangci-lint properly.
I found a way to solve it

ls $(go env GOPATH)/bin

Gives me list of go env paths which includes golangci-lint

So, on doing this.

export PATH=$PATH:$(go env GOPATH)/bin

make lint work and gives me desired results...
So I'm kind of lost here... as I cannot do this in makefile.

Hi @rianhughes I tried to implement golangci-lint.yml file I am facing few problems make lint gives me this error

golangci-lint not installed. Run 'make install-golangci-lint' to install it.
make: *** [Makefile:26: lint] Error 1

Even though I have installed golangci-lint properly. I found a way to solve it

ls $(go env GOPATH)/bin

Gives me list of go env paths which includes golangci-lint

So, on doing this.

export PATH=$PATH:$(go env GOPATH)/bin

make lint work and gives me desired results... So I'm kind of lost here... as I cannot do this in makefile.

Maybe including the following in the Makefile would help?

GOPATH := $(shell go env GOPATH)
GOBIN := $(GOPATH)/bin

install-golangci-lint:
@export PATH=$(GOBIN):$$PATH &&
go install github.com/golangci/golangci-lint/cmd/[email protected]

Hi @rianhughes
Its not working for me...
I have to manually run export PATH=$PATH:$(go env GOPATH)/bin in terminal to make it (make lint) work. Cause makefile is unable to add GOBIN to PATH
Can you please see whether its the same in your end or not??

Hi @rianhughes
Did you find time to review this PR??