Add Go 1.20 support

[austinvazquez]

• Add Go 1.20 to GitHub actions matrix
• Updates actions/setup-go from v1 to v3. (Resolves deprecation warnings in GitHub actions around Node 12. Reference here)

Signed-off-by: Austin Vazquez macedonv@amazon.com

[austinvazquez]

arf; looks like some changes are needed (see failures)

Added a little extra scope to resolve the errors. Took the opportunity to update the lint tooling from golint to golangci-lint and resolved new errors. Included here since we have the hood open but could split this into a seperate commit/PR if that helps.

[austinvazquez]

Adding go.mod explicitly to .gitignore is a little odd when I don't think anything here generates it (right?), but I don't care about that enough to block this.

The other combined CI fixes/updates seem fine to include here IMO 👍

Good call out I could have explained that better. CI generates them here and just wanted to be sure I didn't accidentally push those.

[AkihiroSuda]

Why not use go.mod ?

[austinvazquez]

Why not use go.mod ?

I can make this happen. IIRC other specs had them.

[AkihiroSuda]

@austinvazquez ↑ PTAL

[austinvazquez]

Changes look good overall, but they need to be structured.

1. Fix using deprecated ioutil.
2. Switch to using golangci-lint.
3. Modify ci to use matrix of go versions

One other thing, for golangci-lint it is better to use their action directly from .github/workflows rather than through make.

@kolyshkin, let me draft up what golangci-lint through github action would look like and you can lmk your thoughts.

[austinvazquez]

@kolyshkin, I have split commits and changed linting to use github action. PTAL and let me know your thoughts.

[AkihiroSuda]

CI is failing

[austinvazquez]

CI is failing

Oof I've fallen victim to the go.mod issue. Moved the hack up to resolve.