Idempotent migrate command (#805)

kvch · web-flow · commit f1c78058f32b · 2025-04-30T12:00:26.000+02:00
This PR makes the command `migrate` idempotent. If there is an active migration, `pgroll` does not return an error. The command just returns and lets the user know that there is an active migration. Closes #798
diff --git a/cmd/migrate.go b/cmd/migrate.go
@@ -52,7 +52,8 @@ func migrateCmd() *cobra.Command {
 				return fmt.Errorf("unable to determine active migration period: %w", err)
 			}
 			if active {
-				return fmt.Errorf("migration %q is active and must be completed first", *latestVersion)
+				fmt.Printf("migration %q is active\n", *latestVersion)
+				return nil
 			}
 
 			info, err := os.Stat(migrationsDir)

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,8 @@ func migrateCmd() *cobra.Command {`
`52`	`52`	`return fmt.Errorf("unable to determine active migration period: %w", err)`
`53`	`53`	`}`
`54`	`54`	`if active {`
`55`		`- return fmt.Errorf("migration %q is active and must be completed first", *latestVersion)`
	`55`	`+ fmt.Printf("migration %q is active\n", *latestVersion)`
	`56`	`+ return nil`
`56`	`57`	`}`
`57`	`58`
`58`	`59`	`info, err := os.Stat(migrationsDir)`