should we be using go lint & go vet

I wasn't aware until recently that these were things. I tried running them on our code and they complain about a number of things. This issue is a reminder for someone (probably me) to review what things they check for, see if their complaints are configurable and possibly update our code & workflow to include verifying clean output for new commits.