Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(poetry): normalize package name #19807

Merged
merged 12 commits into from
Jan 14, 2023
Merged

fix(poetry): normalize package name #19807

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
22f6b42
fix(poetry): normalize package name
dave-powell-kensho Jan 11, 2023
6f6a34b
lowercase depName in normalization
dave-powell-kensho Jan 12, 2023
68c8742
set packageName to normalized name instead of depName
dave-powell-kensho Jan 12, 2023
a78e876
Merge branch 'main' into poetry-pep503-normalize
dave-powell-kensho Jan 12, 2023
71892ab
only set packageName if different from depName
dave-powell-kensho Jan 13, 2023
f90fa7f
Merge branch 'main' into poetry-pep503-normalize
rarkins Jan 13, 2023
da69e8f
fix expected test output
dave-powell-kensho Jan 13, 2023
d450027
Merge branch 'main' into poetry-pep503-normalize
rarkins Jan 13, 2023
e2ad1cf
fix order of expected test output
dave-powell-kensho Jan 13, 2023
90252a3
fix eslint errors
dave-powell-kensho Jan 13, 2023
934e488
lint-fix
rarkins Jan 14, 2023
58b86fb
Merge branch 'main' into poetry-pep503-normalize
rarkins Jan 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
8 changes: 5 additions & 3 deletions lib/modules/manager/poetry/extract.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,8 @@ function extractFromSection(
continue;
}

let pep503NormalizeRegex = /[-_.]+/g;
dave-powell-kensho marked this conversation as resolved.
Show resolved Hide resolved
let depNameNormalized = str.replace(pep503NormalizeRegex, "-");
let skipReason: SkipReason | null = null;
let currentValue = sectionContent[depName];
let nestedVersion = false;
Expand All @@ -79,14 +81,14 @@ function extractFromSection(
}
}
const dep: PackageDependency = {
depName,
depName: depNameNormalized,
dave-powell-kensho marked this conversation as resolved.
Show resolved Hide resolved
depType,
currentValue,
managerData: { nestedVersion },
datasource: PypiDatasource.id,
};
if (depName in poetryLockfile) {
dep.lockedVersion = poetryLockfile[depName];
if (depNameNormalized in poetryLockfile) {
dep.lockedVersion = poetryLockfile[depNameNormalized];
}
if (skipReason) {
dep.skipReason = skipReason;
Expand Down