resolve comments

Signed-off-by: Billy Zha <[email protected]>

internal/credential/store.go

@@ -22,21 +22,18 @@ import (
 // NewStore generates a store based on the passed-in config file paths.
 func NewStore(configPaths ...string) (credentials.Store, error) {
 	opts := credentials.StoreOptions{AllowPlaintextPut: true}
-	var store credentials.Store
-	var err error
 	if len(configPaths) == 0 {
 		// use default docker config file path
-		store, err = credentials.NewStoreFromDocker(opts)
-	} else {
-		var stores []credentials.Store
-		for _, config := range configPaths {
-			store, err := credentials.NewStore(config, opts)
-			if err != nil {
-				return nil, err
-			}
-			stores = append(stores, store)
+		return credentials.NewStoreFromDocker(opts)
+	}
+
+	var stores []credentials.Store
+	for _, config := range configPaths {
+		store, err := credentials.NewStore(config, opts)
+		if err != nil {
+			return nil, err
 		}
-		store, err = credentials.NewStoreWithFallbacks(stores[0], stores[1:]...), nil
+		stores = append(stores, store)
 	}
-	return store, err
+	return credentials.NewStoreWithFallbacks(stores[0], stores[1:]...), nil
 }