Remove golangci-lint parameters which are unsupported

[ksamoray]

skip-pkg-cache and skip-build-cache has been removed in v6.x version of golangci-lint, and therefore removed from config. See here:
https://github.com/golangci/golangci-lint-action/blob/master/README.md#compatibility

This will suppress the warning below from golangci-lint execution:

Unexpected input(s) 'skip-pkg-cache', 'skip-build-cache', valid inputs are ['version', 'install-mode', 'working-directory', 'github-token', 'only-new-issues', 'skip-cache', 'skip-save-cache', 'problem-matchers', 'args', 'cache-invalidation-interval']

[tenthirtyam]

LGTM. Those must have snuck back in on the rebase yesterday.

[ksamoray]

@tenthirtyam @annakhm I also consider disabling cache on setup-go due to the following warning:

Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2

and per the following issue:
actions/setup-go#403

What's your opinion about?

[tenthirtyam]

Interesting! I've not seen that happen in other projects but I'd support adding cache: false to the with section.

[tenthirtyam]

LGTM, Kobi!

[annakhm]

@tenthirtyam @annakhm I also consider disabling cache on setup-go due to the following warning:

Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2

and per the following issue: actions/setup-go#403

What's your opinion about?

We've seen jobs time out, but I'm not sure if disabling cache will increase those occurrences. We can certainly try