fix: error message when trying to migrate a migrated config.

The check was done in the wrong place, so it was not triggered
when the config was already migrated.

Another error message was reported about jsonschema validation issues.
Because a v2 config file does not validate against the v1 schema.

That was misleading.

Author: ccoVeille

pkg/commands/migrate.go

@@ -81,10 +81,6 @@ func newMigrateCommand(log logutils.Log, info BuildInfo) *migrateCommand {
 }
 
 func (c *migrateCommand) execute(_ *cobra.Command, _ []string) error {
-	if c.cfg.Version != "" {
-		return fmt.Errorf("configuration version is already set: %s", c.cfg.Version)
-	}
-
 	srcPath := c.viper.ConfigFileUsed()
 	if srcPath == "" {
 		c.log.Warnf("No config file detected")
@@ -151,6 +147,10 @@ func (c *migrateCommand) preRunE(cmd *cobra.Command, _ []string) error {
 		os.Exit(exitcodes.NoConfigFileDetected)
 	}
 
+	if c.cfg.Version != "" {
+		return fmt.Errorf("configuration version is already set: %s", c.cfg.Version)
+	}
+
 	c.log.Infof("Validating v1 configuration file: %s", usedConfigFile)
 
 	err := validateConfiguration("https://golangci-lint.run/jsonschema/golangci.v1.jsonschema.json", usedConfigFile)