History

Fix migration log message showing 'undefined' when no toMigrationInclusive is provided

Cursor Agent committed 7 months ago

Commit 919971a1 parent 30038501 file · +5 −1Browse files

Unified · Split

apps/server/src/persistence/Migrations.ts+5−1
@@ -90,7 +90,11 @@ export interface RunMigrationsOptions { · 89 unmodified lines
90 */90 */
91export const runMigrations = ({ toMigrationInclusive }: RunMigrationsOptions = {}) =>91export const runMigrations = ({ toMigrationInclusive }: RunMigrationsOptions = {}) =>
92 Effect.gen(function* () {92 Effect.gen(function* () {
93 yield* Effect.log(`Running migrations 1 through ${toMigrationInclusive}...`);93 yield* Effect.log(
94 toMigrationInclusive !== undefined
95 ? `Running migrations 1 through ${toMigrationInclusive}...`
96 : "Running migrations...",
97 );
94 const executedMigrations = yield* run({ loader: makeMigrationLoader(toMigrationInclusive) });98 const executedMigrations = yield* run({ loader: makeMigrationLoader(toMigrationInclusive) });
95 yield* Effect.log("Migrations ran successfully");99 yield* Effect.log("Migrations ran successfully");
96 return executedMigrations;100 return executedMigrations;

Keyboard shortcuts

General

Show keyboard shortcuts
?
Focus the current search
/

Issues and pull requests

Open filters
f

Issues, pull requests and Inbox

Select next item
j↓
Select previous item
k↑
Open selected item
Enter
Dismiss filters or clear selection
Esc

Repository code

Go to file
t