Update to go 1.23.x

.github/workflows/ci.yml

@@ -17,7 +17,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v4
       with:
-        go-version: '1.22.x'
+        go-version: '1.23.x'
 
     - name: Build
       run: make chronoctl

.golangci.yml

@@ -73,19 +73,19 @@ linters-settings:
     # minimal occurrences count to trigger, 3 by default
     min-occurrences: 3
   depguard:
-    list-type: blacklist
-    include-go-root: false
-    packages:
-      - hash/fnv
-      - github.com/sirupsen/logrus
-      - github.com/tj/assert
-      - github.com/golang/protobuf/jsonpb
-    packages-with-error-messages:
-      # specify an error message to output when a blacklisted package is used
-      hash/fnv: "use github.com/cespare/xxhash/v2"
-      github.com/sirupsen/logrus: "logging is allowed only by logutils.Log"
-      github.com/tj/assert: "use github.com/stretchr/testify/assert"
-      github.com/golang/protobuf/jsonpb: "replace with github.com/gogo/protobuf/jsonpb"
+    rules:
+      main:
+        deny:
+          - pkg: "hash/fnv"
+            desc: "use github.com/cespare/xxhash/v2"
+          - pkg: "github.com/sirupsen/logrus"
+            desc: "logging is allowed only by logutils.Log"
+          - pkg: "github.com/tj/assert"
+            desc: "use github.com/stretchr/testify/assert"
+          - pkg: "github.com/golang/protobuf/jsonpb"
+            desc: "replace with github.com/gogo/protobuf/jsonpb"
+          - pkg: "golang.org/x/net/context"
+            desc: "replace with context"
   misspell:
     # Correct spellings using locale preferences for US or UK.
     # Default is to use a neutral variety of English.

Makefile

@@ -86,7 +86,7 @@ chronoctl:
 go-version-check:
 	# make sure you're running the right version of Go, otherwise builds/codegen/tests
 	# may have inconsistent results that are hard to debug.
-	go version | grep go1.22 || (echo "Error: you must be running go1.22.x" && exit 1)
+	go version | grep go1.23 || (echo "Error: you must be running go1.23.x" && exit 1)
 
 .PHONY: release
 release:

README.md

@@ -64,7 +64,7 @@ Chronosphere:
 
 ## Building
 
-To build chronoctl, you must have Go 1.22 installed. A download link can be found at
+To build chronoctl, you must have Go 1.23 installed. A download link can be found at
 [go.dev](https://go.dev/doc/install).
 
 Run `make chronoctl` to build the binary. It will be found under

go.mod

@@ -1,6 +1,6 @@
 module github.com/chronosphereio/chronoctl-core
 
-go 1.22
+go 1.23
 
 require (
 	github.com/alecthomas/participle/v2 v2.1.0

src/cmd/apply.go

@@ -163,7 +163,7 @@ func (o *ApplyCommand) Run(stdout io.Writer, stderr io.Writer) error {
 		if dryRun {
 			dryRunPrefix = "[dry run] "
 		}
-		fmt.Fprintf(stdout, "%sApplying %s ", dryRunPrefix, obj.Description())
+		fmt.Fprintf(stdout, "%sApplying %s ", dryRunPrefix, obj.Description()) //nolint:errcheck
 
 		if err := o.Applier(o, obj, stdout, stderr); err != nil {
 			if errors.Is(err, ErrNoApplyMapping) {
@@ -199,7 +199,7 @@ func DefaultApplier(o *ApplyCommand, obj types.Object, out, err io.Writer) error
 		return err
 	}
 
-	fmt.Fprintf(out, "(applied successfully)\n")
+	fmt.Fprintf(out, "(applied successfully)\n") //nolint:errcheck
 	return nil
 }
 

src/cmd/pkg/auth/loginserver.go

@@ -149,7 +149,7 @@ func (s *loginServer) login(ctx context.Context, stdin io.Reader, stdout, stderr
 			return nil
 		}
 		if err := setDefaultOrg(s.store, s.org); err != nil {
-			fmt.Fprintf(stderr, "Failed to set default organization: %s\n", err)
+			fmt.Fprintf(stderr, "Failed to set default organization: %s\n", err) //nolint:errcheck
 		}
 		return nil
 	case err := <-errCh: