fix: add more logging on cli failure

droidsolutions · Sep 24, 2020 · 0d9e739 · 0d9e739
1 parent e6b729d
commit 0d9e739
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 1 deletion.
diff --git a/lib/fail.js b/lib/fail.js
@@ -6,6 +6,13 @@ module.exports = async (pluginConfig, context) => {
       stdio: "inherit",
     });
   } catch (err) {
-    throw new Error(`Unable to use Sentry CLI: ${err.message}`);
+    context.logger.error("fail", err);
+
+    if (pluginConfig.allowSentryFailure) {
+      context.logger.log(`Sentry publish failed, but this is allowed by config. Err: ${err.message}`);
+      return;
+    }
+
+    throw new SemanticReleaseError(`Unable to use Sentry CLI: ${err.message}`, "ESENTRYCLIFAILURE", err);
   }
 };
diff --git a/lib/prepare.js b/lib/prepare.js
@@ -7,6 +7,8 @@ module.exports = async (pluginConfig, context) => {
       stdio: "inherit",
     });
   } catch (err) {
+    context.logger.error("fail", err);
+
     if (pluginConfig.allowSentryFailure) {
       context.logger.log(`Sentry verify failed, but this is allowed by config. Err: ${err.message}`);
       return;

diff --git a/lib/success.js b/lib/success.js
@@ -6,6 +6,8 @@ module.exports = async (pluginConfig, context) => {
       stdio: "inherit",
     });
   } catch (err) {
+    context.logger.error("fail", err);
+
     if (pluginConfig.allowSentryFailure) {
       context.logger.log(`Sentry success/finalize failed, but this is allowed by config. Err: ${err.message}`);
       return;