Andreas PR comments

Author: jeroenvervaeke

cmd/atlas/atlas.go

@@ -50,7 +50,7 @@ To learn more, see our documentation: https://www.mongodb.com/docs/atlas/cli/sta
 
 // loadConfig reads in config file and ENV variables if set.
 func loadConfig() (*config.Profile, error) {
-	configStore, initErr := config.LoadViperStore(afero.NewOsFs())
+	configStore, initErr := config.NewViperStore(afero.NewOsFs())
 
 	if initErr != nil {
 		return nil, fmt.Errorf("error loading config: %w. Please run `atlas auth login` to reconfigure your profile", initErr)

internal/config/viper_store.go

@@ -29,14 +29,15 @@ import (
 	"github.com/spf13/viper"
 )
 
+// ViperConfigStore implements the config.Store interface
 type ViperConfigStore struct {
 	viper     viper.Viper
 	configDir string
 	fs        afero.Fs
 }
 
-// / ViperConfigStore specific methods
-func LoadViperStore(fs afero.Fs) (*ViperConfigStore, error) {
+// ViperConfigStore specific methods
+func NewViperStore(fs afero.Fs) (*ViperConfigStore, error) {
 	configDir, err := CLIConfigHome()
 	if err != nil {
 		return nil, err
@@ -95,7 +96,7 @@ func (s *ViperConfigStore) Filename() string {
 	return ViperConfigStoreFilename(s.configDir)
 }
 
-/// ConfigStore implementation
+// ConfigStore implementation
 
 func (s *ViperConfigStore) Save() error {
 	exists, err := afero.DirExists(s.fs, s.configDir)