ignore 404s in dry-run

github · Nov 1, 2023 · 4998f97 · 4998f97
1 parent 8ec4892
commit 4998f97
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/lib/plugins/diffable.js b/lib/plugins/diffable.js
@@ -153,10 +153,16 @@ module.exports = class Diffable extends ErrorStash {
         }
         return Promise.all(changes)
       }).catch(e => {
-        this.logError(`Error ${e} in ${this.constructor.name} for repo: ${JSON.stringify(this.repo)} entries ${JSON.stringify(this.entries)}`)
         if (this.nop) {
-          resArray.push(new NopCommand(this.constructor.name, this.repo, null, `error ${e} in ${this.constructor.name} for repo: ${JSON.stringify(this.repo)} entries ${JSON.stringify(this.entries)}`, 'ERROR'))
-          return Promise.resolve(resArray)
+          if (e.status === 404) {
+            // Ignore 404s which can happen in dry-run as the repo may not exist.
+            return Promise.resolve(resArray)
+          } else {
+            resArray.push(new NopCommand(this.constructor.name, this.repo, null, `error ${e} in ${this.constructor.name} for repo: ${JSON.stringify(this.repo)} entries ${JSON.stringify(this.entries)}`, 'ERROR'))
+            return Promise.resolve(resArray)
+          }
+        } else {
+          this.logError(`Error ${e} in ${this.constructor.name} for repo: ${JSON.stringify(this.repo)} entries ${JSON.stringify(this.entries)}`)
         }
       })
     }