fix: no tracking on failed deploys

forcedotcom · Apr 13, 2023 · 380d7f8 · 380d7f8
1 parent 317208d
commit 380d7f8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/sourceTracking.ts b/src/sourceTracking.ts
@@ -661,13 +661,13 @@ export class SourceTracking extends AsyncCreatable {
       // yes, the post hooks really have different payloads!
       lifecycle.on('scopedPostDeploy', async (e: ScopedPostDeploy) => {
         this.logger.debug('received scopedPostDeploy event');
-        if (e.orgId === this.orgId) {
+        if (e.orgId === this.orgId && e.deployResult.response.success) {
           await this.updateTrackingFromDeploy(e.deployResult);
         }
       });
       lifecycle.on('scopedPostRetrieve', async (e: ScopedPostRetrieve) => {
         this.logger.debug('received scopedPostRetrieve event');
-        if (e.orgId === this.orgId) {
+        if (e.orgId === this.orgId && e.retrieveResult.response.success) {
           await this.updateTrackingFromRetrieve(e.retrieveResult);
         }
       });