Skip to content

Commit

Permalink
fix(pdm): ignore build requirements when update lock file (#28946)
Browse files Browse the repository at this point in the history
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
  • Loading branch information
huxuan and viceice authored May 10, 2024
1 parent 0f7e4c2 commit aa05c66
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions lib/modules/manager/pep621/processors/pdm.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,7 @@ describe('modules/manager/pep621/processors/pdm', () => {
depType: depTypes.pdmDevDependencies,
},
{ depName: 'group3/dep8', depType: depTypes.pdmDevDependencies },
{ depName: 'dep9', depType: depTypes.buildSystemRequires },
];
const result = await processor.updateArtifacts(
{
Expand Down
4 changes: 4 additions & 0 deletions lib/modules/manager/pep621/processors/pdm.ts
Original file line number Diff line number Diff line change
Expand Up @@ -187,6 +187,10 @@ function generateCMDs(updatedDeps: Upgrade[]): string[] {
);
break;
}
case depTypes.buildSystemRequires:
// build requirements are not locked in the lock files, no need to update.
// Reference: https://github.com/pdm-project/pdm/discussions/2869
break;
default: {
addPackageToCMDRecord(packagesByCMD, pdmUpdateCMD, dep.packageName!);
}
Expand Down

0 comments on commit aa05c66

Please sign in to comment.