lint: remove outdated nolint directives
What does this MR do?
golangci-lint was recently bumped to 1.43.0
It seems those issues from the bodyclose
linter are no longer reported (probably a false positive in older version of the linter).
I've also added the nolintlint
linter to make sure unused //nolint
directives are removed
Closes #606 (closed)
Closes #557 (closed)
TODO
-
I added the Changelog
trailer (e.g.Changelog: feature
) to the commits that need to be included in the changelog -
I added unit tests or they are not required -
I added acceptance tests or they are not required -
I added documentation (or it's not required) -
I followed code review guidelines -
I followed Go Style guidelines
Edited by feistel