From 62b7144ab95b10eb221098c9f063bf00e94d60eb Mon Sep 17 00:00:00 2001 From: Rhys Arkins <rhys@arkins.net> Date: Fri, 7 Jul 2023 20:26:33 +0200 Subject: [PATCH] fix(logs): warn when Package lookup failures occur (#23247) --- lib/workers/repository/errors-warnings.ts | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/workers/repository/errors-warnings.ts b/lib/workers/repository/errors-warnings.ts index d06f2bb1c7..516c74a5aa 100644 --- a/lib/workers/repository/errors-warnings.ts +++ b/lib/workers/repository/errors-warnings.ts @@ -55,6 +55,9 @@ function getDepWarnings( } } } + if (warnings.length) { + logger.warn({ warnings, files: warningFiles }, 'Package lookup failures'); + } return { warnings, warningFiles }; } @@ -70,7 +73,6 @@ export function getDepWarningsOnboardingPR( if (!warnings.length) { return ''; } - logger.debug({ warnings, warningFiles }, 'Found package lookup warnings'); warningText = emojify( `\n---\n\n### :warning: Dependency Lookup Warnings :warning:\n\n` ); @@ -90,7 +92,7 @@ export function getDepWarningsPR( config: RenovateConfig, dependencyDashboard?: boolean ): string { - const { warnings, warningFiles } = getDepWarnings(packageFiles); + const { warnings } = getDepWarnings(packageFiles); if (config.suppressNotifications?.includes('dependencyLookupWarnings')) { return ''; } @@ -98,7 +100,6 @@ export function getDepWarningsPR( if (!warnings.length) { return ''; } - logger.debug({ warnings, warningFiles }, 'Found package lookup warnings'); warningText = emojify( `\n---\n\n### :warning: Dependency Lookup Warnings :warning:\n\n` ); @@ -130,7 +131,6 @@ export function getDepWarningsDashboard( .map((dep) => '`' + dep + '`') .join(', '); - logger.debug({ warnings, warningFiles }, 'Found package lookup warnings'); let warningText = emojify( `\n---\n\n### :warning: Dependency Lookup Warnings :warning:\n\n` ); -- GitLab