fix(Analytics): Ensure to send payload when having all meta

serverless · Nov 3, 2020 · d2b378b · d2b378b
1 parent ed328f1
commit d2b378b
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/lib/plugins/deploy/deploy.js b/lib/plugins/deploy/deploy.js
@@ -114,14 +114,14 @@ class Deploy {
             });
           }
 
-          this.deferredBackendNotificationRequest = analyticsReport(
-            Object.assign(generateAnalyticsPayload(this.serverless), { command: 'deploy' })
-          );
-
-          if (!this.options.package && !this.serverless.service.package.path) {
-            return this.serverless.pluginManager.spawn('package');
-          }
-          return null;
+          return (!this.options.package && !this.serverless.service.package.path
+            ? BbPromise.resolve(this.serverless.pluginManager.spawn('package'))
+            : BbPromise.resolve()
+          ).finally(() => {
+            this.deferredBackendNotificationRequest = analyticsReport(
+              Object.assign(generateAnalyticsPayload(this.serverless), { command: 'deploy' })
+            );
+          });
         }),
       'after:deploy:finalize': () => {
         if (!this.deferredBackendNotificationRequest) return null;