Skip to content
Snippets Groups Projects
Unverified Commit 33e3bf7c authored by Michael Kriese's avatar Michael Kriese Committed by GitHub
Browse files

refactor: move nested function to module scope (#17361)


Co-authored-by: default avatarRhys Arkins <rhys@arkins.net>
parent 57ade2b3
No related branches found
No related tags found
No related merge requests found
......@@ -20,6 +20,27 @@ export const validMatchFields = [
type ValidMatchFields = typeof validMatchFields[number];
function updateDependency(
dependency: PackageDependency,
field: ValidMatchFields,
value: string
): void {
switch (field) {
case 'registryUrl':
// check if URL is valid and pack inside an array
try {
const url = new URL(value).toString();
dependency.registryUrls = [url];
} catch (err) {
logger.warn({ value }, 'Invalid regex manager registryUrl');
}
break;
default:
dependency[field] = value;
break;
}
}
export function createDependency(
extractionTemplate: ExtractionTemplate,
config: CustomExtractConfig,
......@@ -28,30 +49,13 @@ export function createDependency(
const dependency = dep ?? {};
const { groups, replaceString } = extractionTemplate;
function updateDependency(field: ValidMatchFields, value: string): void {
switch (field) {
case 'registryUrl':
// check if URL is valid and pack inside an array
try {
const url = new URL(value).toString();
dependency.registryUrls = [url];
} catch (err) {
logger.warn({ value }, 'Invalid regex manager registryUrl');
}
break;
default:
dependency[field] = value;
break;
}
}
for (const field of validMatchFields) {
const fieldTemplate = `${field}Template` as keyof RegexManagerTemplates;
const tmpl = config[fieldTemplate];
if (tmpl) {
try {
const compiled = template.compile(tmpl, groups, false);
updateDependency(field, compiled);
updateDependency(dependency, field, compiled);
} catch (err) {
logger.warn(
{ template: tmpl },
......@@ -60,7 +64,7 @@ export function createDependency(
return null;
}
} else if (groups[field]) {
updateDependency(field, groups[field]);
updateDependency(dependency, field, groups[field]);
}
}
dependency.replaceString = replaceString;
......
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