fix(pino): show correct statuses for the "list" and "new" commands

aboviq · Feb 12, 2024 · 1065322 · 1065322
1 parent 17feb2d
commit 1065322
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 2 deletions.
diff --git a/.changeset/tender-timers-remember.md b/.changeset/tender-timers-remember.md
@@ -0,0 +1,5 @@
+---
+'@emigrate/reporter-pino': patch
+---
+
+Show correct status for migrations for the "list" and "new" commands
diff --git a/packages/reporter-pino/src/index.ts b/packages/reporter-pino/src/index.ts
@@ -116,12 +116,26 @@ class PinoReporter implements Required<EmigrateReporter> {
   }
 
   onMigrationStart(migration: MigrationMetadata): Awaitable<void> {
-    const status = this.#command === 'up' ? 'running' : 'removing';
+    let status = 'running';
+
+    if (this.#command === 'remove') {
+      status = 'removing';
+    } else if (this.#command === 'new') {
+      status = 'creating';
+    }
+
     this.#logger.info({ migration: migration.relativeFilePath }, `${migration.name} (${status})`);
   }
 
   onMigrationSuccess(migration: MigrationMetadataFinished): Awaitable<void> {
-    const status = this.#command === 'up' ? 'done' : 'removed';
+    let status = 'done';
+
+    if (this.#command === 'remove') {
+      status = 'removed';
+    } else if (this.#command === 'new') {
+      status = 'created';
+    }
+
     this.#logger.info({ migration: migration.relativeFilePath }, `${migration.name} (${status})`);
   }