fix: linter warnings in new Golang CI version (#2883)

Signed-off-by: Philip Laine <[email protected]>

.golangci.yaml

@@ -23,6 +23,7 @@ linters-settings:
       - shadow
       - fieldalignment
       - unusedwrite
+      - printf
   nolintlint:
     require-specific: true
   goheader:

src/extensions/bigbang/test/bigbang_test.go

@@ -47,7 +47,7 @@ func TestMain(m *testing.M) {
 	zarf = fmt.Sprintf("./%s", test.GetCLIName())
 
 	// Run the tests
-	m.Run()
+	os.Exit(m.Run())
 }
 
 func TestReleases(t *testing.T) {

src/pkg/lint/validate.go

@@ -71,7 +71,7 @@ const (
 func ValidatePackage(pkg v1alpha1.ZarfPackage) error {
 	var err error
 	if pkg.Kind == v1alpha1.ZarfInitConfig && pkg.Metadata.YOLO {
-		err = errors.Join(err, fmt.Errorf(PkgValidateErrInitNoYOLO))
+		err = errors.Join(err, errors.New(PkgValidateErrInitNoYOLO))
 	}
 	for _, constant := range pkg.Constants {
 		if varErr := constant.Validate(); varErr != nil {
@@ -84,16 +84,16 @@ func ValidatePackage(pkg v1alpha1.ZarfPackage) error {
 	if pkg.Metadata.YOLO {
 		for _, component := range pkg.Components {
 			if len(component.Images) > 0 {
-				err = errors.Join(err, fmt.Errorf(PkgValidateErrYOLONoOCI))
+				err = errors.Join(err, errors.New(PkgValidateErrYOLONoOCI))
 			}
 			if len(component.Repos) > 0 {
-				err = errors.Join(err, fmt.Errorf(PkgValidateErrYOLONoGit))
+				err = errors.Join(err, errors.New(PkgValidateErrYOLONoGit))
 			}
 			if component.Only.Cluster.Architecture != "" {
-				err = errors.Join(err, fmt.Errorf(PkgValidateErrYOLONoArch))
+				err = errors.Join(err, errors.New(PkgValidateErrYOLONoArch))
 			}
 			if len(component.Only.Cluster.Distros) > 0 {
-				err = errors.Join(err, fmt.Errorf(PkgValidateErrYOLONoDistro))
+				err = errors.Join(err, errors.New(PkgValidateErrYOLONoDistro))
 			}
 		}
 	}
@@ -220,12 +220,12 @@ func validateAction(action v1alpha1.ZarfComponentAction) error {
 
 		// Validate only cluster or network, not both
 		if action.Wait.Cluster != nil && action.Wait.Network != nil {
-			err = errors.Join(err, fmt.Errorf(PkgValidateErrActionClusterNetwork))
+			err = errors.Join(err, errors.New(PkgValidateErrActionClusterNetwork))
 		}
 
 		// Validate at least one of cluster or network
 		if action.Wait.Cluster == nil && action.Wait.Network == nil {
-			err = errors.Join(err, fmt.Errorf(PkgValidateErrActionClusterNetwork))
+			err = errors.Join(err, errors.New(PkgValidateErrActionClusterNetwork))
 		}
 	}
 
@@ -243,7 +243,7 @@ func validateReleaseName(chartName, releaseName string) (err error) {
 
 	// Check if the final releaseName is empty and return an error if so
 	if releaseName == "" {
-		err = fmt.Errorf(errChartReleaseNameEmpty)
+		err = errors.New(errChartReleaseNameEmpty)
 		return
 	}
 

src/pkg/packager/prepare.go

@@ -6,6 +6,7 @@ package packager
 
 import (
 	"context"
+	"errors"
 	"fmt"
 	"os"
 	"path/filepath"
@@ -364,7 +365,7 @@ func (p *Packager) findImages(ctx context.Context) (imgMap map[string][]string,
 			}
 			errMsg += fmt.Sprintf("the following images errored on cosign lookups: %s", erroredCosignLookups)
 		}
-		return imagesMap, fmt.Errorf(errMsg)
+		return imagesMap, errors.New(errMsg)
 	}
 
 	return imagesMap, nil