Investigate use of golangci-lint

[JamesMurkin]

https://golangci-lint.run/

This tool is a metalinter. Running it locally shows up quite a few things we should fix:

• Unused code
• Unused variables

It would be worth putting this in our CI, once we move to github actions.