Turn on revive
linter in golangci config
#442
Labels
devops
Improvements to DevOps (e.g. GitHub actions, linting, etc.)
enhancement
New feature or request
good first issue
Good for newcomers
help wanted
Extra attention is needed
low priority
Would be nice to fix, but doesn't have to happen right now/there are more important things
stale
Describe the desired feature/enhancement
Seems like
golangci
dropped support forgolint
a few versions ago. IMO, we should definitely have some sort of linting enabled!Is your feature request related to a problem?
The problem: I cannot stand unlinted codebases!
Describe the solution you'd like
Add
revive
, a drop-in replacement forgolint
, to the list of enabled linters.golangci.yml
.This will require going through the entire codebase and fixing all of the errors until
just lint
is happy. Runninggolangci-lint -c .golangci.yml run --fix
will take you part of the way there. It would probably also be good to add another entry to thejustfile
calledlint-fix
that runsgolangci-lint -c .golangci.yml run --fix
, or allow the user to pass additional arguments to thelint
target (e.g.just lint --fix
).Describe alternatives you've considered (optional)
We could just leave the linter off, but that would make me sad :(
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: