refactor: move config path logic to separate func

Author: muthukrishnan24

config/config.go

@@ -49,34 +49,48 @@ var allRules = []lint.Rule{
 	&rule.DescriptionMinLenRule{},
 }
 
-// GetConfig returns conf and follows the belwo precedence
+// GetConfig returns conf
+func GetConfig(flagConfPath string) (*lint.Config, error) {
+	confFilePath, useDefault, err := GetConfigPath(flagConfPath)
+	if err != nil {
+		return nil, err
+	}
+
+	if useDefault {
+		return defConf, nil
+	}
+
+	conf, err := Parse(confFilePath)
+	if err != nil {
+		return nil, err
+	}
+	return conf, nil
+}
+
+// GetConfigPath returns config file path, follwing below
 // 	1. check for conf in current directory
 // 	2. check for conf flag
 // 	3. load default conf
-func GetConfig(confFilePath string) (*lint.Config, error) {
+func GetConfigPath(confFilePath string) (string, bool, error) {
 	// get current directory
 	currentDir, err := os.Getwd()
 	if err != nil {
-		return nil, err
+		return "", false, err
 	}
 
 	// check if conf file exists in current directory
 	currentDirConf := filepath.Join(currentDir, ConfFileName)
 	if _, err1 := os.Stat(currentDirConf); !os.IsNotExist(err1) {
-		confFilePath = currentDirConf
+		return currentDirConf, false, nil
 	}
 
-	// if confFilePath empty, means no config in current directory or config flag is empty
+	// if confFilePath empty,
+	// means no config in current directory or config flag is empty
+	// use default config
 	if confFilePath == "" {
-		return defConf, nil
+		return "", true, nil
 	}
-
-	confFilePath = filepath.Clean(confFilePath)
-	conf, err := Parse(confFilePath)
-	if err != nil {
-		return nil, err
-	}
-	return conf, nil
+	return filepath.Clean(confFilePath), false, nil
 }
 
 // GetFormatter returns the formatter as defined in conf