chore: upgrade to Go 1.23.1

.github/workflows/lint.yml

@@ -22,10 +22,9 @@ jobs:
             go.sum
       - uses: golangci/golangci-lint-action@v6.1.0
         with:
-          version: v1.59.1
+          version: v1.61.0
           args: --timeout 10m
           github-token: ${{ secrets.github_token }}
-          skip-pkg-cache: true
         if: env.GIT_DIFF
 
   # hadolint lints the Dockerfile

.golangci.yml

@@ -4,7 +4,7 @@ run:
 
 linters:
   enable:
-    - exportloopref
+    - copyloopvar
     - gofumpt
     - misspell
     - nakedret

.goreleaser.yaml

@@ -108,7 +108,7 @@ archives:
 checksum:
   name_template: "checksums.txt"
 snapshot:
-  name_template: "{{ incpatch .Version }}-next"
+  version_template: "{{ incpatch .Version }}-next"
 changelog:
   sort: asc
   filters:

Dockerfile

@@ -4,7 +4,7 @@
 #
 # Separating the builder and runtime image allows the runtime image to be
 # considerably smaller because it doesn't need to have Golang installed.
-ARG BUILDER_IMAGE=docker.io/golang:1.22.6-alpine3.19
+ARG BUILDER_IMAGE=docker.io/golang:1.23.1-alpine3.20
 ARG RUNTIME_IMAGE=docker.io/alpine:3.19
 ARG TARGETOS
 ARG TARGETARCH

Makefile

@@ -7,7 +7,9 @@ DOCKER_PROTO_BUILDER := docker run -v $(shell pwd):/workspace --workdir /workspa
 PROJECTNAME=$(shell basename "$(PWD)")
 HTTPS_GIT := https://github.com/celestiaorg/celestia-app.git
 PACKAGE_NAME          := github.com/celestiaorg/celestia-app/v3
-GOLANG_CROSS_VERSION  ?= v1.22.6
+# Before upgrading the GOLANG_CROSS_VERSION, please verify that a Docker image exists with the new tag.
+# See https://github.com/goreleaser/goreleaser-cross/pkgs/container/goreleaser-cross
+GOLANG_CROSS_VERSION  ?= v1.23.1
 
 # process linker flags
 ldflags = -X github.com/cosmos/cosmos-sdk/version.Name=celestia-app \
@@ -246,4 +248,4 @@ enable-bbr:
 	else \
 	    echo "BBR is already enabled."; \
 	fi
-.PHONY: enable-bbr
+.PHONY: enable-bbr

README.md

@@ -35,7 +35,7 @@ node            |  |                               |  |
 
 ### Source
 
-1. [Install Go](https://go.dev/doc/install) 1.22.6
+1. [Install Go](https://go.dev/doc/install) 1.23.1
 1. Clone this repo
 1. Install the celestia-app CLI
 
@@ -134,7 +134,7 @@ This repo contains multiple go modules. When using it, rename `go.work.example`
 
 ### Tools
 
-1. Install [golangci-lint](https://golangci-lint.run/welcome/install) 1.59.1
+1. Install [golangci-lint](https://golangci-lint.run/welcome/install) 1.61.0
 1. Install [markdownlint](https://github.com/DavidAnson/markdownlint) 0.39.0
 1. Install [hadolint](https://github.com/hadolint/hadolint)
 1. Install [yamllint](https://yamllint.readthedocs.io/en/stable/quickstart.html)

app/test/priority_test.go

@@ -80,7 +80,6 @@ func (s *PriorityTestSuite) TestPriorityByGasPrice() {
 	wg := &sync.WaitGroup{}
 	for _, accName := range s.accountNames {
 		wg.Add(1)
-		accName := accName // new variable per iteration
 		go func() {
 			defer wg.Done()
 			// ensure that it is greater than the min gas price

docker/Dockerfile_txsim

@@ -1,5 +1,5 @@
 # Stage 1: generate celestia-appd binary
-FROM --platform=$BUILDPLATFORM docker.io/golang:1.22.6-alpine3.19 as builder
+FROM --platform=$BUILDPLATFORM docker.io/golang:1.23.1-alpine3.20 as builder
 
 ARG TARGETOS
 ARG TARGETARCH

go.mod

@@ -1,6 +1,6 @@
 module github.com/celestiaorg/celestia-app/v3
 
-go 1.22.6
+go 1.23.1
 
 require (
 	cosmossdk.io/errors v1.0.1

go.work.example

@@ -1,4 +1,4 @@
-go 1.22.6
+go 1.23.1
 
 use (
 	.

pkg/da/data_availability_header_test.go

@@ -88,7 +88,6 @@ func TestExtendShares(t *testing.T) {
 	}
 
 	for _, tt := range tests {
-		tt := tt
 		_, err := ExtendShares(tt.shares)
 		if tt.expectedErr {
 			require.NotNil(t, err)
@@ -122,7 +121,6 @@ func TestDataAvailabilityHeaderProtoConversion(t *testing.T) {
 	}
 
 	for _, tt := range tests {
-		tt := tt
 		pdah, err := tt.dah.ToProto()
 		require.NoError(t, err)
 		resDah, err := DataAvailabilityHeaderFromProto(pdah)
@@ -203,7 +201,6 @@ func Test_DAHValidateBasic(t *testing.T) {
 	}
 
 	for _, tt := range tests {
-		tt := tt
 		err := tt.dah.ValidateBasic()
 		if tt.expectErr {
 			require.True(t, strings.Contains(err.Error(), tt.errStr), tt.name)

test/interchain/go.mod

@@ -1,6 +1,6 @@
 module github.com/celestiaorg/celestia-app/test/interchain
 
-go 1.22.6
+go 1.23.1
 
 require (
 	cosmossdk.io/math v1.3.0

tools/blocketa/README.md

@@ -15,4 +15,4 @@ arrivalTime: 2024-08-28 17:24:23.483542677 +0000 UTC
 ```
 
 > [!NOTE]
-> The block time is currently hard-coded. If you're running this for a network with a different block time, you'll need to update the `blockTime` constant in the main.go file. You can use [https://www.mintscan.io/celestia/block](https://www.mintscan.io/celestia/block/) or the blocktime tool.
+> The block time is currently hard-coded. If you're running this for a network with a different block time, you'll need to update the `blockTime` constant in the main.go file. You can use [https://www.mintscan.io/celestia/block](https://www.mintscan.io/celestia/block/) or the blocktime tool.

x/blob/client/testutil/integration_test.go

@@ -143,7 +143,6 @@ func (s *IntegrationTestSuite) TestSubmitPayForBlob() {
 	}
 
 	for _, tc := range testCases {
-		tc := tc
 		require.NoError(s.ctx.WaitForNextBlock())
 		s.Run(tc.name, func() {
 			cmd := paycli.CmdPayForBlob()

x/blobstream/keeper/keeper_data_commitment.go

@@ -109,7 +109,7 @@ func (k Keeper) GetLatestDataCommitment(ctx sdk.Context) (types.DataCommitment,
 			return types.DataCommitment{}, err
 		}
 		if !found {
-			return types.DataCommitment{}, errors.Wrapf(types.ErrAttestationNotFound, fmt.Sprintf("nonce %d", i))
+			return types.DataCommitment{}, errors.Wrapf(types.ErrAttestationNotFound, "nonce %d", i)
 		}
 		dcc, ok := att.(*types.DataCommitment)
 		if !ok {
@@ -136,7 +136,7 @@ func (k Keeper) HasDataCommitmentInStore(ctx sdk.Context) (bool, error) {
 			return false, err
 		}
 		if !found {
-			return false, errors.Wrapf(types.ErrAttestationNotFound, fmt.Sprintf("nonce %d", i))
+			return false, errors.Wrapf(types.ErrAttestationNotFound, "nonce %d", i)
 		}
 		_, ok := att.(*types.DataCommitment)
 		if !ok {

x/blobstream/keeper/keeper_valset_test.go

@@ -57,7 +57,6 @@ func TestCurrentValsetNormalization(t *testing.T) {
 		},
 	}
 	for msg, spec := range specs {
-		spec := spec
 		t.Run(msg, func(t *testing.T) {
 			input, ctx := testutil.SetupTestChain(t, spec.srcPowers)
 			r, err := input.BlobstreamKeeper.GetCurrentValset(ctx)

x/mint/client/testutil/suite_test.go

@@ -68,8 +68,6 @@ func (s *IntegrationTestSuite) TestGetCmdQueryInflationRate() {
 	}
 
 	for _, tc := range testCases {
-		tc := tc
-
 		s.Run(tc.name, func() {
 			cmd := cli.GetCmdQueryInflationRate()
 
@@ -103,8 +101,6 @@ func (s *IntegrationTestSuite) TestGetCmdQueryAnnualProvisions() {
 
 	expectedAnnualProvision := mint.InitialInflationRateAsDec().MulInt(sdk.NewInt(testnode.DefaultInitialBalance))
 	for _, tc := range testCases {
-		tc := tc
-
 		s.Run(tc.name, func() {
 			cmd := cli.GetCmdQueryAnnualProvisions()
 			out, err := clitestutil.ExecTestCLICmd(s.cctx.Context, cmd, tc.args)
@@ -134,8 +130,6 @@ func (s *IntegrationTestSuite) TestGetCmdQueryGenesisTime() {
 	}
 
 	for _, tc := range testCases {
-		tc := tc
-
 		s.Run(tc.name, func() {
 			cmd := cli.GetCmdQueryGenesisTime()
 			out, err := clitestutil.ExecTestCLICmd(s.cctx.Context, cmd, tc.args)

x/mint/simulation/decoder_test.go

@@ -51,7 +51,6 @@ func TestDecodeStore(t *testing.T) {
 	}
 
 	for i, tt := range tests {
-		i, tt := i, tt
 		t.Run(tt.name, func(t *testing.T) {
 			if tt.expectPanic {
 				require.Panics(t, func() { decoder(kvPairs.Pairs[i], kvPairs.Pairs[i]) }, tt.name)