Merge pull request #236 from ulucinar/bump-go1.20

Bump Go version to v1.20 in go.mod

.github/workflows/ci.yml

@@ -10,8 +10,8 @@ on:
 
 env:
   # Common versions
-  GO_VERSION: '1.19'
-  GOLANGCI_VERSION: 'v1.50.0'
+  GO_VERSION: '1.20'
+  GOLANGCI_VERSION: 'v1.53.3'
   DOCKER_BUILDX_VERSION: 'v0.8.2'
 
   # Common users. We can't run a step 'if secrets.AWS_USR != ""' but we can run
@@ -48,7 +48,7 @@ jobs:
           submodules: true
 
       - name: Setup Go
-        uses: actions/setup-go@v2
+        uses: actions/setup-go@v3
         with:
           go-version: ${{ env.GO_VERSION }}
 
@@ -81,10 +81,9 @@ jobs:
       # this action because it leaves 'annotations' (i.e. it comments on PRs to
       # point out linter violations).
       - name: Lint
-        uses: golangci/golangci-lint-action@v2
+        uses: golangci/golangci-lint-action@639cd343e1d3b897ff35927a75193d57cfcba299 # v3
         with:
           version: ${{ env.GOLANGCI_VERSION }}
-          skip-go-installation: true
 
   check-diff:
     runs-on: ubuntu-20.04

Makefile

@@ -4,8 +4,8 @@
 PROJECT_NAME := upjet
 PROJECT_REPO := github.com/upbound/$(PROJECT_NAME)
 
-GOLANGCILINT_VERSION ?= 1.50.0
-GO_REQUIRED_VERSION ?= 1.19
+GOLANGCILINT_VERSION ?= 1.53.3
+GO_REQUIRED_VERSION ?= 1.20
 
 PLATFORMS ?= linux_amd64 linux_arm64
 # -include will silently skip missing files, which allows us

go.mod

@@ -1,6 +1,6 @@
 module github.com/upbound/upjet
 
-go 1.19
+go 1.20
 
 require (
 	github.com/antchfx/htmlquery v1.2.4

pkg/migration/filesystem.go

@@ -152,7 +152,7 @@ func (ft *FileSystemTarget) Put(o UnstructuredWithMetadata) error {
 
 		defer f.Close() //nolint:errcheck
 
-		if _, err = f.WriteString(fmt.Sprintf("\n---\n\n%s", string(b))); err != nil {
+		if _, err = fmt.Fprintf(f, "\n---\n\n%s", string(b)); err != nil {
 			return errors.Wrap(err, "cannot write file")
 		}
 	} else {