linting: make changes from new linting version

With the new version of the linter, new code has been flagged. Address the lints where possible and ignore the non relivant.
coreos · Oct 11, 2024 · d4700f1 · d4700f1
1 parent 8d49bb2
commit d4700f1
Show file tree

Hide file tree

Showing 6 changed files with 8 additions and 8 deletions.
diff --git a/internal/exec/config_fetcher.go b/internal/exec/config_fetcher.go
@@ -100,7 +100,7 @@ func (f *ConfigFetcher) fetchReferencedConfig(cfgRef types.Resource) (types.Conf
 		return types.Config{}, err
 	}
 	var headers http.Header
-	if cfgRef.HTTPHeaders != nil && len(cfgRef.HTTPHeaders) > 0 {
+	if len(cfgRef.HTTPHeaders) > 0 {
 		headers, err = cfgRef.HTTPHeaders.Parse()
 		if err != nil {
 			return types.Config{}, err

diff --git a/internal/exec/engine.go b/internal/exec/engine.go
@@ -121,7 +121,7 @@ func (e Engine) Run(stageName string) error {
 		return err
 	}
 
-	e.Logger.PushPrefix(stageName)
+	e.Logger.PushPrefix(stageName) //nolint:govet
 	defer e.Logger.PopPrefix()
 
 	fullConfig := latest.Merge(baseConfig, latest.Merge(systemBaseConfig, cfg))

diff --git a/internal/exec/stages/files/files.go b/internal/exec/stages/files/files.go
@@ -164,7 +164,7 @@ func (s *stage) relabel(paths ...string) {
 // relabelFiles relabels all the files that were marked for relabeling using
 // the libselinux APIs.
 func (s *stage) relabelFiles() error {
-	if s.toRelabel == nil || len(s.toRelabel) == 0 {
+	if len(s.toRelabel) == 0 {
 		return nil
 	}
 

diff --git a/internal/exec/util/file.go b/internal/exec/util/file.go
@@ -78,7 +78,7 @@ func newFetchOp(l *log.Logger, node types.Node, contents types.Resource) (FetchO
 	}
 
 	var headers http.Header
-	if contents.HTTPHeaders != nil && len(contents.HTTPHeaders) > 0 {
+	if len(contents.HTTPHeaders) > 0 {
 		headers, err = contents.HTTPHeaders.Parse()
 		if err != nil {
 			return FetchOp{}, err

diff --git a/internal/main.go b/internal/main.go
@@ -89,7 +89,7 @@ func ignitionMain() {
 	logger := log.New(flags.logToStdout)
 	defer logger.Close()
 
-	logger.Info(version.String)
+	logger.Info(version.String) //nolint:govet
 	logger.Info("Stage: %v", flags.stage)
 
 	platformConfig := platform.MustGet(flags.platform.String())
@@ -157,7 +157,7 @@ func ignitionApplyMain() {
 	logger := log.New(true)
 	defer logger.Close()
 
-	logger.Info(version.String)
+	logger.Info(version.String) //nolint:govet
 
 	var blob []byte
 	var err error
@@ -219,7 +219,7 @@ func ignitionRmCfgMain() {
 	logger := log.New(flags.logToStdout)
 	defer logger.Close()
 
-	logger.Info(version.String)
+	logger.Info(version.String) //nolint:govet
 
 	platformConfig := platform.MustGet(flags.platform)
 	fetcher, err := platformConfig.NewFetcher(&logger)

diff --git a/internal/resource/http.go b/internal/resource/http.go
@@ -171,7 +171,7 @@ func (f *Fetcher) getCABlob(ca types.Resource) ([]byte, error) {
 	}
 
 	var headers http.Header
-	if ca.HTTPHeaders != nil && len(ca.HTTPHeaders) > 0 {
+	if len(ca.HTTPHeaders) > 0 {
 		headers, err = ca.HTTPHeaders.Parse()
 		if err != nil {
 			return nil, err