refactor

provider/appconfig/appconfig.go

@@ -107,7 +107,7 @@
 	for {
 		select {
 		case <-ticker.C:
-			out, err := a.load(ctx)
+			values, err := a.load(ctx)
 			if err != nil {
 				a.logger.WarnContext(
 					ctx, "Error when reloading from AWS AppConfig",
@@ -120,8 +120,8 @@
 				continue
 			}
 
-			if out != nil {
-				onChange(out)
+			if values != nil {
+				onChange(values)
 			}
 		case <-ctx.Done():
 			return nil
@@ -208,16 +208,14 @@
 	c.clientOnce.Do(func() {
 		if reflect.ValueOf(c.config).IsZero() {
 			if c.config, err = config.LoadDefaultConfig(ctx); err != nil {
+				err = fmt.Errorf("load default AWS config: %w", err)
+
 				return
 			}
 		}
 
 		c.client = appconfigdata.NewFromConfig(c.config)
 	})
 
-	if err != nil {
-		return nil, fmt.Errorf("load client: %w", err)
-	}
-
-	return c.client, nil
+	return c.client, err
 }

provider/flag/option.go

@@ -42,7 +42,7 @@ func WithDelimiter(delimiter string) Option {
 	}
 }
 
-// WithNameSplitter provides the function used to split environment variable names into nested keys.
+// WithNameSplitter provides the function used to split flag names into nested keys.
 // If it returns an nil/[]string{}/[]string{""}, the variable will be ignored.
 //
 // For example, with the default splitter, an flag name like "parent.child.key"

provider/pflag/option.go

@@ -43,7 +43,7 @@ func WithDelimiter(delimiter string) Option {
 	}
 }
 
-// WithNameSplitter provides the function used to split environment variable names into nested keys.
+// WithNameSplitter provides the function used to split flag names into nested keys.
 // If it returns an nil/[]string{}/[]string{""}, the variable will be ignored.
 //
 // For example, with the default splitter, an flag name like "parent.child.key"