added todo for future check

ariym · Nov 18, 2023 · 8b79c97 · 8b79c97
1 parent 00b9288
commit 8b79c97
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/download.ts b/src/download.ts
@@ -80,9 +80,11 @@ export default async function downloadModel() {
     // windows .cmd version
     if(process.platform === 'win32') scriptPath = "download-ggml-model.cmd";
 
-    // todo: check if windows or unix to run bat command or .sh command
     shell.exec(`${scriptPath} ${modelName}`);
 
+    // TODO: add check in case download-ggml-model doesn't return a successful download.
+    // to prevent continuing to compile; that makes it harder for user to see which script failed.
+
     console.log("[whisper-node] Attempting to compile model...");
 
     // move up directory, run make in whisper.cpp