Skip to content

Commit

Permalink
Run full set of checks using Go 1.23 (#350)
Browse files Browse the repository at this point in the history
Some of our static check tools were too old and didn't work with Go >=
1.20.

This PR bumps the versions of those tools so they can work with the
current latest Go version (1.23) and runs the full set of CI checks on
that version.
  • Loading branch information
gpassini authored Nov 30, 2024
1 parent 366ca4f commit 5a89c1a
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 12 deletions.
16 changes: 7 additions & 9 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,15 +13,8 @@ jobs:
working_directory: ~/repo
docker:
- image: cimg/go:1.21
steps:
- checkout
- run:
name: Run tests and linters
command: |
make ci
steps: *simple_job_steps

# TODO: Need updates to some static analyzer tools to support 1.22.
# After those are updated, move the full linting from 1.21 to this latest release.
build-1-22:
working_directory: ~/repo
docker:
Expand All @@ -32,7 +25,12 @@ jobs:
working_directory: ~/repo
docker:
- image: cimg/go:1.23
steps: *simple_job_steps
steps:
- checkout
- run:
name: Run tests and linters
command: |
make ci
workflows:
pr-build-test:
Expand Down
6 changes: 3 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ ci: deps checkgofmt checkgenerate vet staticcheck ineffassign predeclared test

.PHONY: deps
deps:
go get -d -v -t ./...
go get -v -t ./...
go mod tidy

.PHONY: updatedeps
Expand Down Expand Up @@ -68,7 +68,7 @@ vet:

.PHONY: staticcheck
staticcheck:
@go install honnef.co/go/tools/cmd/staticcheck@v0.4.3
@go install honnef.co/go/tools/cmd/staticcheck@v0.5.1
staticcheck ./...

.PHONY: ineffassign
Expand All @@ -78,7 +78,7 @@ ineffassign:

.PHONY: predeclared
predeclared:
@go install github.com/nishanths/predeclared@5f2f810c9ae6
@go install github.com/nishanths/predeclared@245576f9a85c96ea16c750df3887f1d827f01e9c
predeclared ./...

# Intentionally omitted from CI, but target here for ad-hoc reports.
Expand Down

0 comments on commit 5a89c1a

Please sign in to comment.