chore: reorganize methods

Author: ldez

pkg/config/loader.go

@@ -71,75 +71,6 @@ func (l *Loader) Load() error {
 	return nil
 }
 
-func (l *Loader) handleEnableOnlyOption() error {
-	only, err := l.fs.GetStringSlice("enable-only")
-	if err != nil {
-		return err
-	}
-
-	if len(only) > 0 {
-		l.cfg.Linters = Linters{
-			Enable:     only,
-			DisableAll: true,
-		}
-	}
-
-	return nil
-}
-
-func (l *Loader) handleGoVersion() {
-	if l.cfg.Run.Go == "" {
-		l.cfg.Run.Go = detectGoVersion()
-	}
-
-	l.cfg.LintersSettings.Govet.Go = l.cfg.Run.Go
-
-	l.cfg.LintersSettings.ParallelTest.Go = l.cfg.Run.Go
-
-	if l.cfg.LintersSettings.Gofumpt.LangVersion == "" {
-		l.cfg.LintersSettings.Gofumpt.LangVersion = l.cfg.Run.Go
-	}
-
-	trimmedGoVersion := trimGoVersion(l.cfg.Run.Go)
-
-	l.cfg.LintersSettings.Gocritic.Go = trimmedGoVersion
-
-	// staticcheck related linters.
-	if l.cfg.LintersSettings.Staticcheck.GoVersion == "" {
-		l.cfg.LintersSettings.Staticcheck.GoVersion = trimmedGoVersion
-	}
-	if l.cfg.LintersSettings.Gosimple.GoVersion == "" {
-		l.cfg.LintersSettings.Gosimple.GoVersion = trimmedGoVersion
-	}
-	if l.cfg.LintersSettings.Stylecheck.GoVersion != "" {
-		l.cfg.LintersSettings.Stylecheck.GoVersion = trimmedGoVersion
-	}
-}
-
-func (l *Loader) handleDeprecation() {
-	if len(l.cfg.Run.SkipFiles) > 0 {
-		l.warn("The configuration option `run.skip-files` is deprecated, please use `issues.exclude-files`.")
-		l.cfg.Issues.ExcludeFiles = l.cfg.Run.SkipFiles
-	}
-
-	if len(l.cfg.Run.SkipDirs) > 0 {
-		l.warn("The configuration option `run.skip-dirs` is deprecated, please use `issues.exclude-dirs`.")
-		l.cfg.Issues.ExcludeDirs = l.cfg.Run.SkipDirs
-	}
-
-	// The 2 options are true by default.
-	if !l.cfg.Run.UseDefaultSkipDirs {
-		l.warn("The configuration option `run.skip-dirs-use-default` is deprecated, please use `issues.exclude-dirs-use-default`.")
-	}
-	l.cfg.Issues.UseDefaultExcludeDirs = l.cfg.Run.UseDefaultSkipDirs && l.cfg.Issues.UseDefaultExcludeDirs
-
-	// The 2 options are false by default.
-	if l.cfg.Run.ShowStats {
-		l.warn("The configuration option `run.show-stats` is deprecated, please use `output.show-stats`")
-	}
-	l.cfg.Output.ShowStats = l.cfg.Run.ShowStats || l.cfg.Output.ShowStats
-}
-
 func (l *Loader) setConfigFile() error {
 	configFile, err := l.evaluateOptions()
 	if err != nil {
@@ -319,6 +250,75 @@ func (l *Loader) appendStringSlice(name string, current *[]string) {
 	}
 }
 
+func (l *Loader) handleGoVersion() {
+	if l.cfg.Run.Go == "" {
+		l.cfg.Run.Go = detectGoVersion()
+	}
+
+	l.cfg.LintersSettings.Govet.Go = l.cfg.Run.Go
+
+	l.cfg.LintersSettings.ParallelTest.Go = l.cfg.Run.Go
+
+	if l.cfg.LintersSettings.Gofumpt.LangVersion == "" {
+		l.cfg.LintersSettings.Gofumpt.LangVersion = l.cfg.Run.Go
+	}
+
+	trimmedGoVersion := trimGoVersion(l.cfg.Run.Go)
+
+	l.cfg.LintersSettings.Gocritic.Go = trimmedGoVersion
+
+	// staticcheck related linters.
+	if l.cfg.LintersSettings.Staticcheck.GoVersion == "" {
+		l.cfg.LintersSettings.Staticcheck.GoVersion = trimmedGoVersion
+	}
+	if l.cfg.LintersSettings.Gosimple.GoVersion == "" {
+		l.cfg.LintersSettings.Gosimple.GoVersion = trimmedGoVersion
+	}
+	if l.cfg.LintersSettings.Stylecheck.GoVersion != "" {
+		l.cfg.LintersSettings.Stylecheck.GoVersion = trimmedGoVersion
+	}
+}
+
+func (l *Loader) handleDeprecation() {
+	if len(l.cfg.Run.SkipFiles) > 0 {
+		l.warn("The configuration option `run.skip-files` is deprecated, please use `issues.exclude-files`.")
+		l.cfg.Issues.ExcludeFiles = l.cfg.Run.SkipFiles
+	}
+
+	if len(l.cfg.Run.SkipDirs) > 0 {
+		l.warn("The configuration option `run.skip-dirs` is deprecated, please use `issues.exclude-dirs`.")
+		l.cfg.Issues.ExcludeDirs = l.cfg.Run.SkipDirs
+	}
+
+	// The 2 options are true by default.
+	if !l.cfg.Run.UseDefaultSkipDirs {
+		l.warn("The configuration option `run.skip-dirs-use-default` is deprecated, please use `issues.exclude-dirs-use-default`.")
+	}
+	l.cfg.Issues.UseDefaultExcludeDirs = l.cfg.Run.UseDefaultSkipDirs && l.cfg.Issues.UseDefaultExcludeDirs
+
+	// The 2 options are false by default.
+	if l.cfg.Run.ShowStats {
+		l.warn("The configuration option `run.show-stats` is deprecated, please use `output.show-stats`")
+	}
+	l.cfg.Output.ShowStats = l.cfg.Run.ShowStats || l.cfg.Output.ShowStats
+}
+
+func (l *Loader) handleEnableOnlyOption() error {
+	only, err := l.fs.GetStringSlice("enable-only")
+	if err != nil {
+		return err
+	}
+
+	if len(only) > 0 {
+		l.cfg.Linters = Linters{
+			Enable:     only,
+			DisableAll: true,
+		}
+	}
+
+	return nil
+}
+
 func (l *Loader) warn(format string) {
 	if l.cfg.InternalTest || l.cfg.InternalCmdTest || os.Getenv(logutils.EnvTestRun) == "1" {
 		return