From bc04da5e41dd5f7aab1266340f04e8bb46355ae1 Mon Sep 17 00:00:00 2001 From: Mariusz Nowak Date: Mon, 23 Dec 2019 11:49:06 +0100 Subject: [PATCH] refactor(User Config): Show config save error only with SLS_DEBUG=* It's to not confuse users with warnings which do not affect how application works Fixes #7077 --- lib/utils/config/index.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/lib/utils/config/index.js b/lib/utils/config/index.js index 47b8815e8..0ba706cfe 100644 --- a/lib/utils/config/index.js +++ b/lib/utils/config/index.js @@ -23,10 +23,12 @@ function storeConfig(config) { try { writeFileAtomic.sync(serverlessrcPath, JSON.stringify(config, null, 2)); } catch (error) { - if (process.env.SLS_DEBUG) process.stdout.write(`${chalk.red(error.stack)}\n`); - process.stdout.write( - `Serverless: ${chalk.red(`Unable to store serverless config due to ${error.code} error`)}\n` - ); + if (process.env.SLS_DEBUG) { + process.stdout.write(`${chalk.red(error.stack)}\n`); + process.stdout.write( + `Serverless: ${chalk.red(`Unable to store serverless config due to ${error.code} error`)}\n` + ); + } try { return JSON.parse(readFileSync(serverlessrcPath)); } catch (readError) {