From facf46c84e0b7833c396f1503ce4645c99bd19b2 Mon Sep 17 00:00:00 2001 From: Rhys Arkins <rhys@arkins.net> Date: Sat, 12 Sep 2020 07:28:12 +0200 Subject: [PATCH] fix: treat config validation errors as enabled (#7255) --- lib/workers/repository/result.ts | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/lib/workers/repository/result.ts b/lib/workers/repository/result.ts index 6db4f90227..68819ee0c6 100644 --- a/lib/workers/repository/result.ts +++ b/lib/workers/repository/result.ts @@ -1,5 +1,7 @@ import { RenovateConfig } from '../../config'; import { + CONFIG_SECRETS_EXPOSED, + CONFIG_VALIDATION, MANAGER_NO_PACKAGE_FILES, REPOSITORY_ACCESS_FORBIDDEN, REPOSITORY_ARCHIVED, @@ -12,6 +14,7 @@ import { REPOSITORY_RENAMED, REPOSITORY_UNINITIATED, } from '../../constants/error-messages'; +import { logger } from '../../logger'; type ProcessStatus = 'disabled' | 'enabled' | 'onboarding' | 'unknown'; export interface ProcessResult { @@ -36,15 +39,17 @@ export function processResult( REPOSITORY_UNINITIATED, REPOSITORY_EMPTY, ]; + const enabledStatuses = [CONFIG_SECRETS_EXPOSED, CONFIG_VALIDATION]; let status: ProcessStatus; // istanbul ignore next if (disabledStatuses.includes(res)) { status = 'disabled'; - } else if (config.repoIsOnboarded) { + } else if (enabledStatuses.includes(res) || config.repoIsOnboarded) { status = 'enabled'; } else if (config.repoIsOnboarded === false) { status = 'onboarding'; } else { + logger.debug({ res }, 'Unknown res'); status = 'unknown'; } return { res, status }; -- GitLab