Chore: update to go 1.20 and grafana-plugin-ci 1.8.1

[yuri-tceretian]

What this PR does / why we need it:
Updates minimal version of Go to 1.20. Because the current version 1.6.1 of grafana-plugin-ci uses Go 1.19 I had to bump the version to 1.8.1

[xnyo]

The new container image has a more recent version of golangci-lint as well, which has some new checks regarding unused arguments. This causes the CI pipeline to fail at the lint step.

I think I have a WIP branch somewhere where I started addressing those issues, if you want you can remove the linter causing the issue (revive) from .golangci.toml and I can take care of fixing those linter issues later 😃

@wbrowne are you okay with that?

You can disable those checks by deleting this line:

grafana-plugin-sdk-go/.golangci.toml

Line 50 in 7dd9759

"revive"

Then CI should pass

[wbrowne]

The new container image has a more recent version of golangci-lint as well, which has some new checks regarding unused arguments. This causes the CI pipeline to fail at the lint step.

I think I have a WIP branch somewhere where I started addressing those issues, if you want you can remove the linter causing the issue (revive) from .golangci.toml and I can take care of fixing those linter issues later 😃

@wbrowne are you okay with that?

You can disable those checks by deleting this line:

grafana-plugin-sdk-go/.golangci.toml

Line 50 in 7dd9759

"revive"

Then CI should pass

Yeah is probably fine 👍

[yuri-tceretian]

I fixed linter failures in the last commit. The changes are straightforward. If you think there is something else can be done, or it can be done differently, I am fine removing the last commit and keep revive rule disabled.

[xnyo]

Thank you for addressing the linter issues as well! 🙏

[xnyo]

LGTM!