We should use char(14) for migration pk

The generated primary key required exactly that length.
phogolabs · Jul 27, 2021 · 1bda5df · 1bda5df
1 parent cdfc124
commit 1bda5df
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/sqlmigr/executor.go b/sqlmigr/executor.go
@@ -31,7 +31,7 @@ func (m *Executor) Setup() error {
 	up := &bytes.Buffer{}
 
 	fmt.Fprintln(up, "CREATE TABLE IF NOT EXISTS migrations (")
-	fmt.Fprintln(up, " id          CHAR(15)  NOT NULL PRIMARY KEY,")
+	fmt.Fprintln(up, " id          CHAR(14)  NOT NULL PRIMARY KEY,")
 	fmt.Fprintln(up, " description TEXT      NOT NULL,")
 	fmt.Fprintln(up, " created_at  TIMESTAMP NOT NULL")
 	fmt.Fprintln(up, ");")

diff --git a/sqlmigr/executor_test.go b/sqlmigr/executor_test.go
@@ -54,7 +54,7 @@ var _ = Describe("Executor", func() {
 
 			up := &bytes.Buffer{}
 			fmt.Fprintln(up, "CREATE TABLE IF NOT EXISTS migrations (")
-			fmt.Fprintln(up, " id          CHAR(15)  NOT NULL PRIMARY KEY,")
+			fmt.Fprintln(up, " id          CHAR(14)  NOT NULL PRIMARY KEY,")
 			fmt.Fprintln(up, " description TEXT      NOT NULL,")
 			fmt.Fprintln(up, " created_at  TIMESTAMP NOT NULL")
 			fmt.Fprintln(up, ");")