after review changes

lint/package.go

@@ -187,18 +187,10 @@ func (p *Package) lint(rules []Rule, config Config, failures chan Failure) error
 	var eg errgroup.Group
 	for _, file := range p.files {
 		eg.Go(func() error {
-			err := file.lint(rules, config, failures)
-			if err != nil {
-				return err
-			}
-			return nil
+			return file.lint(rules, config, failures)
 		})
 	}
-	if err := eg.Wait(); err != nil {
-		return err
-	}
-
-	return nil
+	return eg.Wait()
 }
 
 // IsAtLeastGo121 returns true if the Go version for this package is 1.21 or higher, false otherwise

rule/add_constant.go

@@ -272,9 +272,3 @@ func (r *AddConstantRule) configure(arguments lint.Arguments) error {
 
 	return nil
 }
-
-func configOnce(r *AddConstantRule, arguments lint.Arguments) func() error {
-	return sync.OnceValue(func() error {
-		return r.configure(arguments)
-	})
-}

rule/banned_characters.go

@@ -12,17 +12,17 @@ import (
 // BannedCharsRule checks if a file contains banned characters.
 type BannedCharsRule struct {
 	bannedCharList []string
-	
-	configureOnce  sync.Once
+
+	configureOnce sync.Once
 }
 
 const bannedCharsRuleName = "banned-characters"
 
 func (r *BannedCharsRule) configure(arguments lint.Arguments) error {
 	if len(arguments) > 0 {
-		check := checkNumberOfArguments(1, arguments, bannedCharsRuleName)
-		if check != nil {
-			return check
+		err := checkNumberOfArguments(1, arguments, bannedCharsRuleName)
+		if err != nil {
+			return err
 		}
 		list, err := r.getBannedCharsList(arguments)
 		if err != nil {

rule/receiver_naming.go

@@ -111,7 +111,7 @@ func (r *ReceiverNamingRule) Apply(file *lint.File, arguments lint.Arguments) ([
 		typeReceiver[recv] = name
 	}
 
-	return failures
+	return failures, nil
 }
 
 // Name returns the rule name.