go version bump to v1.22 (#64)

Author: vladopajic

.github/.testcoverage.yml

@@ -4,3 +4,6 @@ profile: cover.out
 local-prefix: "github.com/vladopajic/go-test-coverage/v2"
 threshold:
   total: 90
+exclude:
+  paths:
+    - main\.go$

.github/workflows/lint.yml

@@ -14,6 +14,6 @@ jobs:
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v3
         with:
-          version: v1.55.2
+          version: v1.56.0
       - name: go mod tidy check
         uses: katexochen/go-tidy-check@v2

Dockerfile

@@ -1,5 +1,5 @@
 # GO_VERSION: go version should match version in go.mod file
-FROM golang:1.21 as builder
+FROM golang:1.22 as builder
 WORKDIR /workspace
 
 COPY go.mod go.mod

Makefile

@@ -1,7 +1,7 @@
 GO ?= go
 GOBIN ?= $$($(GO) env GOPATH)/bin
 GOLANGCI_LINT ?= $(GOBIN)/golangci-lint
-GOLANGCI_LINT_VERSION ?= v1.55.2
+GOLANGCI_LINT_VERSION ?= v1.56.0
 
 .PHONY: get-golangcilint
 get-golangcilint:

go.mod

@@ -1,7 +1,7 @@
 module github.com/vladopajic/go-test-coverage/v2
 
 // GO_VERSION: when changing go version update version in other places
-go 1.21
+go 1.22
 
 require (
 	github.com/alexflint/go-arg v1.4.3

pkg/testcoverage/badge/generate_test.go

@@ -9,11 +9,11 @@ import (
 	. "github.com/vladopajic/go-test-coverage/v2/pkg/testcoverage/badge"
 )
 
+//nolint:govet,paralleltest // false-positive
 func Test_Generate(t *testing.T) {
 	t.Parallel()
 
-	for i := 0; i <= 100; i++ {
-		i := i
+	for i := range 100 {
 		c := strconv.Itoa(i) + "%"
 		t.Run("coverage "+c, func(t *testing.T) {
 			t.Parallel()
@@ -46,7 +46,7 @@ func Test_Color(t *testing.T) {
 	colors := make(map[string]struct{})
 
 	{ // Assert that there are 5 colors for coverage [0-101]
-		for i := 0; i <= 101; i++ {
+		for i := range 101 {
 			color := Color(i)
 			colors[color] = struct{}{}
 		}

pkg/testcoverage/badgestorer/github.go

@@ -41,7 +41,7 @@ func (s *githubStorer) Store(data []byte) (bool, error) {
 			git.Repository,
 			git.FileName,
 			&github.RepositoryContentFileOptions{
-				Message: github.String(fmt.Sprintf("update badge %s", git.FileName)),
+				Message: github.String("update badge " + git.FileName),
 				Content: data,
 				Branch:  &git.Branch,
 				SHA:     sha,

pkg/testcoverage/check_test.go

@@ -130,7 +130,7 @@ func TestCheck(t *testing.T) {
 	})
 }
 
-//nolint:paralleltest // must not be parallel because it uses env
+// must not be parallel because it uses env
 func TestCheckNoParallel(t *testing.T) {
 	if testing.Short() {
 		return

pkg/testcoverage/config.go

@@ -1,6 +1,7 @@
 package testcoverage
 
 import (
+	"errors"
 	"fmt"
 	"os"
 	"reflect"
@@ -13,11 +14,11 @@ import (
 )
 
 var (
-	ErrThresholdNotInRange         = fmt.Errorf("threshold must be in range [0 - 100]")
-	ErrCoverageProfileNotSpecified = fmt.Errorf("coverage profile file not specified")
-	ErrRegExpNotValid              = fmt.Errorf("regular expression is not valid")
-	ErrCDNOptionNotSet             = fmt.Errorf("CDN options are not valid")
-	ErrGitOptionNotSet             = fmt.Errorf("git options are not valid")
+	ErrThresholdNotInRange         = errors.New("threshold must be in range [0 - 100]")
+	ErrCoverageProfileNotSpecified = errors.New("coverage profile file not specified")
+	ErrRegExpNotValid              = errors.New("regular expression is not valid")
+	ErrCDNOptionNotSet             = errors.New("CDN options are not valid")
+	ErrGitOptionNotSet             = errors.New("git options are not valid")
 )
 
 type Config struct {
@@ -128,7 +129,7 @@ func (c Config) validateGit() error {
 
 func hasNonEmptyFields(obj any) error {
 	v := reflect.ValueOf(obj)
-	for i := 0; i < v.NumField(); i++ {
+	for i := range v.NumField() {
 		f := v.Field(i)
 
 		if !f.IsZero() { // filed is set

pkg/testcoverage/helpers_test.go

@@ -31,7 +31,7 @@ func randStats(localPrefix string, minc, maxc int) []coverage.Stats {
 	for {
 		pkg := randPackageName(localPrefix)
 
-		for c := rand.Int31n(10); c >= 0; c-- {
+		for range rand.Int31n(10) {
 			total, covered := coverageGen()
 			stat := coverage.Stats{
 				Name:    randFileName(pkg),

pkg/testcoverage/report_test.go

@@ -122,6 +122,7 @@ func Test_ReportForGithubAction(t *testing.T) {
 
 	t.Run("package coverage - pass", func(t *testing.T) {
 		t.Parallel()
+
 		buf := &bytes.Buffer{}
 		cfg := Config{Threshold: Threshold{Package: 10}}
 		statsNoError := randStats(prefix, 10, 100)
@@ -165,7 +166,7 @@ func Test_ReportForGithubAction(t *testing.T) {
 	})
 }
 
-//nolint:paralleltest // must not be parallel because it uses env
+// must not be parallel because it uses env
 func Test_SetGithubActionOutput(t *testing.T) {
 	if testing.Short() {
 		return