diff --git a/.golangci.yml b/.golangci.yml index 9a75b919..f82887de 100644 --- a/.golangci.yml +++ b/.golangci.yml @@ -11,13 +11,11 @@ linters: enable: - gofmt - govet - - deadcode - depguard - goimports - ineffassign - misspell - unused - - varcheck - revive - staticcheck - typecheck diff --git a/Dockerfile b/Dockerfile index f1cc6a16..c8d71191 100644 --- a/Dockerfile +++ b/Dockerfile @@ -1,6 +1,6 @@ # syntax=docker/dockerfile-upstream:1.5.0 -ARG GO_VERSION=1.19 +ARG GO_VERSION=1.20 ARG XX_VERSION=1.1.2 ARG DOCKERD_VERSION=20.10.14 diff --git a/bake/bake_test.go b/bake/bake_test.go index 4311b252..b42a7ee6 100644 --- a/bake/bake_test.go +++ b/bake/bake_test.go @@ -10,8 +10,6 @@ import ( ) func TestReadTargets(t *testing.T) { - t.Parallel() - fp := File{ Name: "config.hcl", Data: []byte(` @@ -35,6 +33,7 @@ target "webapp" { ctx := context.TODO() t.Run("NoOverrides", func(t *testing.T) { + t.Parallel() m, g, err := ReadTargets(ctx, []File{fp}, []string{"webapp"}, nil, nil) require.NoError(t, err) require.Equal(t, 1, len(m)) @@ -50,6 +49,7 @@ target "webapp" { }) t.Run("InvalidTargetOverrides", func(t *testing.T) { + t.Parallel() _, _, err := ReadTargets(ctx, []File{fp}, []string{"webapp"}, []string{"nosuchtarget.context=foo"}, nil) require.NotNil(t, err) require.Equal(t, err.Error(), "could not find any target matching 'nosuchtarget'") @@ -91,6 +91,7 @@ target "webapp" { // building leaf but overriding parent fields t.Run("parent", func(t *testing.T) { + t.Parallel() m, g, err := ReadTargets(ctx, []File{fp}, []string{"webapp"}, []string{ "webDEP.args.VAR_INHERITED=override", "webDEP.args.VAR_BOTH=override", @@ -105,6 +106,7 @@ target "webapp" { }) t.Run("ContextOverride", func(t *testing.T) { + t.Parallel() _, _, err := ReadTargets(ctx, []File{fp}, []string{"webapp"}, []string{"webapp.context"}, nil) require.NotNil(t, err) @@ -116,6 +118,7 @@ target "webapp" { }) t.Run("NoCacheOverride", func(t *testing.T) { + t.Parallel() m, g, err := ReadTargets(ctx, []File{fp}, []string{"webapp"}, []string{"webapp.no-cache=false"}, nil) require.NoError(t, err) require.Equal(t, false, *m["webapp"].NoCache) @@ -124,6 +127,7 @@ target "webapp" { }) t.Run("PullOverride", func(t *testing.T) { + t.Parallel() m, g, err := ReadTargets(ctx, []File{fp}, []string{"webapp"}, []string{"webapp.pull=false"}, nil) require.NoError(t, err) require.Equal(t, false, *m["webapp"].Pull) @@ -132,6 +136,7 @@ target "webapp" { }) t.Run("PatternOverride", func(t *testing.T) { + t.Parallel() // same check for two cases multiTargetCheck := func(t *testing.T, m map[string]*Target, g map[string]*Group, err error) { require.NoError(t, err) diff --git a/bake/hcl_test.go b/bake/hcl_test.go index 56d705ad..ecb0d722 100644 --- a/bake/hcl_test.go +++ b/bake/hcl_test.go @@ -946,7 +946,7 @@ func TestVarUnsupportedType(t *testing.T) { } func ptrstr(s interface{}) *string { - var n *string = nil + var n *string if reflect.ValueOf(s).Kind() == reflect.String { ss := s.(string) n = &ss diff --git a/docker-bake.hcl b/docker-bake.hcl index 644912da..99eaaecc 100644 --- a/docker-bake.hcl +++ b/docker-bake.hcl @@ -1,5 +1,5 @@ variable "GO_VERSION" { - default = "1.19" + default = "1.20" } variable "DOCS_FORMATS" { default = "md" diff --git a/hack/dockerfiles/docs.Dockerfile b/hack/dockerfiles/docs.Dockerfile index c78d0b75..94ab3b1a 100644 --- a/hack/dockerfiles/docs.Dockerfile +++ b/hack/dockerfiles/docs.Dockerfile @@ -1,6 +1,6 @@ # syntax=docker/dockerfile:1.4 -ARG GO_VERSION=1.19 +ARG GO_VERSION=1.20 ARG FORMATS=md,yaml FROM golang:${GO_VERSION}-alpine AS docsgen diff --git a/hack/dockerfiles/generated-files.Dockerfile b/hack/dockerfiles/generated-files.Dockerfile index 85c8729d..cf9a2ce7 100644 --- a/hack/dockerfiles/generated-files.Dockerfile +++ b/hack/dockerfiles/generated-files.Dockerfile @@ -5,7 +5,7 @@ # Copyright The Buildx Authors. # Licensed under the Apache License, Version 2.0 -ARG GO_VERSION="1.19" +ARG GO_VERSION="1.20" ARG PROTOC_VERSION="3.11.4" # protoc is dynamically linked to glibc so can't use alpine base diff --git a/hack/dockerfiles/lint.Dockerfile b/hack/dockerfiles/lint.Dockerfile index bbff367c..5092e85c 100644 --- a/hack/dockerfiles/lint.Dockerfile +++ b/hack/dockerfiles/lint.Dockerfile @@ -1,10 +1,10 @@ # syntax=docker/dockerfile:1.4 -ARG GO_VERSION=1.19 +ARG GO_VERSION=1.20 FROM golang:${GO_VERSION}-alpine RUN apk add --no-cache git gcc musl-dev -RUN wget -O- -nv https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s v1.48.0 +RUN wget -O- -nv https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s v1.51.1 WORKDIR /go/src/github.com/docker/buildx RUN --mount=target=/go/src/github.com/docker/buildx --mount=target=/root/.cache,type=cache \ golangci-lint run diff --git a/hack/dockerfiles/vendor.Dockerfile b/hack/dockerfiles/vendor.Dockerfile index e5e9b7a6..a928e12f 100644 --- a/hack/dockerfiles/vendor.Dockerfile +++ b/hack/dockerfiles/vendor.Dockerfile @@ -1,6 +1,6 @@ # syntax=docker/dockerfile:1.4 -ARG GO_VERSION=1.19 +ARG GO_VERSION=1.20 ARG MODOUTDATED_VERSION=v0.8.0 FROM golang:${GO_VERSION}-alpine AS base