fix TypeScript migration stub after 0.95.0 changes (#4366)

* fix TypeScript migration stub after 0.95.0 changes Release 0.95.0 changed how types have to be imported when using TypeScript. This PR fixes the migration stub so new migrations are created correctly. * Add test for Knex.AlterTableBuilder Co-authored-by: Igor Savin <iselwin@gmail.com>
knex · Mar 11, 2021 · 548471d · 548471d
1 parent 5d1f479
commit 548471d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/migrations/migrate/stub/ts.stub b/lib/migrations/migrate/stub/ts.stub
@@ -2,7 +2,7 @@ import { Knex } from "knex";
 
 <% if (d.tableName) { %>
 export async function up(knex: Knex): Promise<Knex.SchemaBuilder> {
-    return knex.schema.createTable("<%= d.tableName %>", (t: Knex.AlterTableBuilder) => {
+    return knex.schema.createTable("<%= d.tableName %>", (t) => {
         t.increments();
         t.timestamps();
     });

diff --git a/test-tsd/types.test-d.ts b/test-tsd/types.test-d.ts
@@ -24,6 +24,8 @@ expectType<any>(knexCjs({}));
 // eslint-disable-next-line
 expectType<any>(knexCjsNamed({}));
 
+knex({}).schema.createTable("table", (t: Knex.AlterTableBuilder) => {})
+
 const knexInstance = knexDefault(clientConfig);
 
 // ToDo remove this copy-pasted type after we can export it as a named properly