🏃 Update golangci-lint to 1.47.3

[alvaroaleman]

Updates golangci-lint in github actions

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: alvaroaleman

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [alvaroaleman]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[alculquicondor]

It looks like some files where violating the lint

[alvaroaleman]

Seems the latest version wants to gofmt in the (incompatible) go 1.19 style, I went back one version to not have go 1.19 support as the repo is tested with go 1.18, lets see how that goes

[alvaroaleman]

The error assertions in the dynamic rest mappers checkAndReload never worked and fixing them makes the tests fail :/

[alculquicondor]

The error assertions in the dynamic rest mappers checkAndReload never worked

What do you mean? The old code looked fine to me after inspection.

[alculquicondor]

/lgtm
/hold
in case you would like to have a review from an experienced contributor to this repo.

[alvaroaleman]

I think this is fine, as can be seen by the prior failures this has a very good test coverage
/hold cancel