From a2c4ecefda0e0569f570839a03d7e1cc28fe702e Mon Sep 17 00:00:00 2001
From: Rhys Arkins <rhys@arkins.net>
Date: Sun, 4 Mar 2018 07:12:02 +0100
Subject: [PATCH] fix: delete empty errors and warnings from migrated config
---
lib/config/migrate-validate.js | 4 ++--
lib/workers/repository/init/config.js | 2 ++
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/lib/config/migrate-validate.js b/lib/config/migrate-validate.js
index d29592e6e8..ec33e6ef5f 100644
--- a/lib/config/migrate-validate.js
+++ b/lib/config/migrate-validate.js
@@ -18,10 +18,10 @@ function migrateAndValidate(config, input) {
const massagedConfig = configMassage.massageConfig(migratedConfig);
const { warnings, errors } = configValidation.validateConfig(massagedConfig);
// istanbul ignore if
- if (warnings.length) {
+ if (warnings && warnings.length) {
logger.info({ warnings }, 'Found renovate config warnings');
}
- if (errors.length) {
+ if (errors && errors.length) {
logger.info({ errors }, 'Found renovate config errors');
}
massagedConfig.errors = (config.errors || []).concat(errors);
diff --git a/lib/workers/repository/init/config.js b/lib/workers/repository/init/config.js
index 59c8e904f2..b99c3df553 100644
--- a/lib/workers/repository/init/config.js
+++ b/lib/workers/repository/init/config.js
@@ -77,6 +77,8 @@ async function mergeRenovateConfig(config) {
.join(', ');
throw error;
}
+ delete migratedConfig.errors;
+ delete migratedConfig.warnings;
logger.debug({ config: migratedConfig }, 'renovate.json migrated config');
const decryptedConfig = decryptConfig(migratedConfig, config.privateKey);
const resolvedConfig = await presets.resolveConfigPresets(decryptedConfig);
--
GitLab