fix(pep621): handle dependency-groups (PEP 735) in pdm lockfile updat…

…es (#32952)
renovatebot · Dec 6, 2024 · 9f600d1 · 9f600d1
1 parent 5cb9980
commit 9f600d1
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/lib/modules/manager/pep621/processors/pdm.spec.ts b/lib/modules/manager/pep621/processors/pdm.spec.ts
@@ -173,6 +173,11 @@ describe('modules/manager/pep621/processors/pdm', () => {
           managerData: { depGroup: 'group3' },
         },
         { packageName: 'dep9', depType: depTypes.buildSystemRequires },
+        {
+          packageName: 'dep10',
+          depType: depTypes.dependencyGroups,
+          managerData: { depGroup: 'dev' },
+        },
       ];
       const result = await processor.updateArtifacts(
         {
@@ -205,6 +210,9 @@ describe('modules/manager/pep621/processors/pdm', () => {
         {
           cmd: 'pdm update --no-sync --update-eager -dG group3 dep7 dep8',
         },
+        {
+          cmd: 'pdm update --no-sync --update-eager -dG dev dep10',
+        },
       ]);
     });
 
@@ -232,6 +240,10 @@ describe('modules/manager/pep621/processors/pdm', () => {
           packageName: 'dep5',
           depType: depTypes.pdmDevDependencies,
         },
+        {
+          packageName: 'dep10',
+          depType: depTypes.dependencyGroups,
+        },
       ];
       const result = await processor.updateArtifacts(
         {
@@ -244,7 +256,7 @@ describe('modules/manager/pep621/processors/pdm', () => {
       );
       expect(result).toBeNull();
       expect(execSnapshots).toEqual([]);
-      expect(logger.once.warn).toHaveBeenCalledTimes(2);
+      expect(logger.once.warn).toHaveBeenCalledTimes(3);
     });
 
     it('return update on lockfileMaintenance', async () => {

diff --git a/lib/modules/manager/pep621/processors/pdm.ts b/lib/modules/manager/pep621/processors/pdm.ts
@@ -193,6 +193,7 @@ function generateCMDs(updatedDeps: Upgrade<Pep621ManagerData>[]): string[] {
         );
         break;
       }
+      case depTypes.dependencyGroups:
       case depTypes.pdmDevDependencies: {
         if (is.nullOrUndefined(dep.managerData?.depGroup)) {
           logger.once.warn(