Skip to content
Snippets Groups Projects
Unverified Commit 62b7144a authored by Rhys Arkins's avatar Rhys Arkins Committed by GitHub
Browse files

fix(logs): warn when Package lookup failures occur (#23247)

parent 0d6a8882
No related branches found
No related tags found
No related merge requests found
...@@ -55,6 +55,9 @@ function getDepWarnings( ...@@ -55,6 +55,9 @@ function getDepWarnings(
} }
} }
} }
if (warnings.length) {
logger.warn({ warnings, files: warningFiles }, 'Package lookup failures');
}
return { warnings, warningFiles }; return { warnings, warningFiles };
} }
...@@ -70,7 +73,6 @@ export function getDepWarningsOnboardingPR( ...@@ -70,7 +73,6 @@ export function getDepWarningsOnboardingPR(
if (!warnings.length) { if (!warnings.length) {
return ''; return '';
} }
logger.debug({ warnings, warningFiles }, 'Found package lookup warnings');
warningText = emojify( warningText = emojify(
`\n---\n\n### :warning: Dependency Lookup Warnings :warning:\n\n` `\n---\n\n### :warning: Dependency Lookup Warnings :warning:\n\n`
); );
...@@ -90,7 +92,7 @@ export function getDepWarningsPR( ...@@ -90,7 +92,7 @@ export function getDepWarningsPR(
config: RenovateConfig, config: RenovateConfig,
dependencyDashboard?: boolean dependencyDashboard?: boolean
): string { ): string {
const { warnings, warningFiles } = getDepWarnings(packageFiles); const { warnings } = getDepWarnings(packageFiles);
if (config.suppressNotifications?.includes('dependencyLookupWarnings')) { if (config.suppressNotifications?.includes('dependencyLookupWarnings')) {
return ''; return '';
} }
...@@ -98,7 +100,6 @@ export function getDepWarningsPR( ...@@ -98,7 +100,6 @@ export function getDepWarningsPR(
if (!warnings.length) { if (!warnings.length) {
return ''; return '';
} }
logger.debug({ warnings, warningFiles }, 'Found package lookup warnings');
warningText = emojify( warningText = emojify(
`\n---\n\n### :warning: Dependency Lookup Warnings :warning:\n\n` `\n---\n\n### :warning: Dependency Lookup Warnings :warning:\n\n`
); );
...@@ -130,7 +131,6 @@ export function getDepWarningsDashboard( ...@@ -130,7 +131,6 @@ export function getDepWarningsDashboard(
.map((dep) => '`' + dep + '`') .map((dep) => '`' + dep + '`')
.join(', '); .join(', ');
logger.debug({ warnings, warningFiles }, 'Found package lookup warnings');
let warningText = emojify( let warningText = emojify(
`\n---\n\n### :warning: Dependency Lookup Warnings :warning:\n\n` `\n---\n\n### :warning: Dependency Lookup Warnings :warning:\n\n`
); );
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment