From 03d81a2b764cd45fb879f1e908bb6f9e145b7bf6 Mon Sep 17 00:00:00 2001 From: AJ ONeal Date: Wed, 8 Apr 2026 14:38:05 -0600 Subject: [PATCH] fix(sql-migrate): reject explicit 'up 0' as invalid An explicit 0 argument to 'up' should error (like 'down 0' already does), not silently run all pending migrations. Change the guard from < 0 to < 1 to match 'down' behavior. --- cmd/sql-migrate/main.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cmd/sql-migrate/main.go b/cmd/sql-migrate/main.go index 1979567..a38be93 100644 --- a/cmd/sql-migrate/main.go +++ b/cmd/sql-migrate/main.go @@ -344,10 +344,10 @@ func main() { var upN int switch len(leafArgs) { case 0: - // ignore + // no arg: upN stays 0, meaning "all pending" case 1: upN, err = strconv.Atoi(leafArgs[0]) - if err != nil || upN < 0 { + if err != nil || upN < 1 { fmt.Fprintf(os.Stderr, "Error: %s is not a positive number\n", leafArgs[0]) os.Exit(1) }