furthermore, I would recommend using wsl in CI to automatically detect these kinds of errors in the future.
wsl is included in golangci-lint, which is the defacto golang linter, which it seems like you don't use currently.

I prefer to avoid small dependencies. idt this is common enough a bug to justify incorporatingwsl.

I disagree thatgolangci-lint is the defacto go linter.

It'sgoimports,gofmt,golint andgo vet all of which we use and are from the Go team.

1. Well, to be technical,goimports andgofmt are code formatters, not linters. =p
2. golangci-lint includesgoimports,gofmt,golint,govet, as well as various other useful linters, like the previously mentionedwsl.
3. Note thatgolint is nowofficially deprecated and not very many people use it anymore:x/lint: freeze and deprecate golang/go#38968

For my personal Golang projects, I've found a tremendous amount of value in using golanci-lint. If you want to keep it simple and not use it, that's fine - I just thought I would throw it out there for consideration.

golanci-lint has some linters enabled by default and some disabled by default. My typical workflow is that I enable every single linter that it offers. This generates a lot of noise, so then I go through and manually disable the ones that I disagree with, and the ones that are noisy. After that, I am left with a few legitimate issues / bugs in my code, which I clean up, making the whole process worth it. =)

An example yaml config file:
https://github.com/Zamiell/hanabi-live/blob/master/server/.golangci.yml

Well, to be technical, goimports and gofmt are code formatters, not linters. =p

Fair but the line has grown ever thin. code formatting isn't far from automated linting imo.

Note that golint is now officially deprecated and not very many people use it anymore:golang/go#38968

That's wild and definitely puts a dent ingolint for me. I've found it extremely useful, I don't really understand the motivation for removing it in that thread and fully agree withgolang/go#38968 (comment)

For my personal Golang projects, I've found a tremendous amount of value in using golanci-lint. If you want to keep it simple and not use it, that's fine - I just thought I would throw it out there for consideration.

Yea I appreciate the thoughts and discussion but for now I'm definitely in favour of keeping things as minimal as possible.

Perhaps I'll switch the library toStaticcheck which seems like it's highly recommended from that thread and a sort of successor to golint.