CI: add codespell, bump golangci-lint#1281
Conversation
|
ping @kolyshkin |
83b32c4 to
4c838a8
Compare
.github/workflows/build.yml
Outdated
| strategy: | ||
| matrix: | ||
| go: [1.21.x, 1.22.x] | ||
| go: [1.24.x, 1.25.x] |
There was a problem hiding this comment.
GOTOOLCHAIN=local is needed to pin the Go version
There was a problem hiding this comment.
Should we make this <lowest-version-supported>, oldstable, stable ? That also reduces the need to update versions (we'll continue testing some "lowest" version", and the other two test currently supported versions.
There was a problem hiding this comment.
See opencontainers/image-spec#1229 for an example.
There was a problem hiding this comment.
Let me just remove the last commit so we can merge this.
|
Some spelling issues to fix; |
marquiz
left a comment
There was a problem hiding this comment.
👍 LGTM (with linter errors fixed)
Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
Those are (relatively) new, from #1209. It's a pity that it was merged earlier than this PR. Fixed now. |
|
PTAL @sudo-bmitch @thaJeztah |
bump Go to supported versions (1.24.x, 1.25.x).(no longer included)See individual commits for details.