From 420537a64d1b99eae69a6c37a1897c4e09ce1b8e Mon Sep 17 00:00:00 2001 From: "J. Ryan Stinnett" Date: Tue, 25 Feb 2020 11:44:33 +0000 Subject: [PATCH] Relax macOS notarisation check to print a warning This makes it a bit more friendly to build macOS Electron builds without a notarisation config set up by only warning when it is missing. --- scripts/electron_afterSign.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/scripts/electron_afterSign.js b/scripts/electron_afterSign.js index 5952976abd..5e5850b0b5 100644 --- a/scripts/electron_afterSign.js +++ b/scripts/electron_afterSign.js @@ -10,7 +10,10 @@ exports.default = async function(context) { // from the keychain, so we need to get it from the environment. const userId = process.env.NOTARIZE_APPLE_ID; if (userId === undefined) { - throw new Error("User ID not found. Set NOTARIZE_APPLE_ID."); + console.warn( + "Skipping notarisation: User ID not found, set NOTARIZE_APPLE_ID.", + ); + return; } console.log("Notarising macOS app. This may be some time.");