-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Closed
Labels
Description
Hiya! One of the nice features of Visual Studio Code is that it integrates linters very easily.
Unfortunately, checking a few different linters comes up with a slew of issues in this codebase. I've moved to golangci-lint because it's one of the few that supports Go modules.
Here's a snippet of the issues I'm seeing:
Some questions I have:
- Should we resolve these? If so, what would be the best way to PR these changes?
- Should we add the linter(s) to CI to prevent getting into the same scenario in the future?