- debounce manifest change check (#919)

google · Sep 26, 2022 · 7158370 · 7158370
1 parent 2b836a8
commit 7158370
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/src/commands/push.ts b/src/commands/push.ts
@@ -43,6 +43,11 @@ export default async (options: CommandOption): Promise<void> => {
     console.log(LOG.PUSH_WATCH);
     // Debounce calls to push to coalesce 'save all' actions from editors
     const debouncedPushFiles = debounce(async () => {
+      if (!options.force && (await manifestHasChanges(projectSettings)) && !(await confirmManifestUpdate())) {
+        console.log('Stopping push…');
+        return;
+      }
+
       console.log(LOG.PUSHING);
       return pushFiles();
     }, WATCH_DEBOUNCE_MS);
@@ -53,10 +58,6 @@ export default async (options: CommandOption): Promise<void> => {
         return;
       }
       console.log(`\n${LOG.PUSH_WATCH_UPDATED(filePath)}\n`);
-      if (!options.force && (await manifestHasChanges(projectSettings)) && !(await confirmManifestUpdate())) {
-        console.log('Stopping push…');
-        return;
-      }
       return debouncedPushFiles();
     };
     const watcher = chokidar.watch(rootDir, {persistent: true, ignoreInitial: true});