Do not use deprecated poetry --dev option for installing packages (#…

…19980) Closes #19983 https://python-poetry.org/docs/managing-dependencies/#dependency-groups python-poetry/poetry#6475
microsoft · Oct 11, 2022 · af9793f · af9793f
1 parent c6ff83a
commit af9793f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/client/common/installer/poetryInstaller.ts b/src/client/common/installer/poetryInstaller.ts
@@ -70,7 +70,7 @@ export class PoetryInstaller extends ModuleInstaller {
 
     protected async getExecutionInfo(moduleName: string, resource?: InterpreterUri): Promise<ExecutionInfo> {
         const execPath = this.configurationService.getSettings(isResource(resource) ? resource : undefined).poetryPath;
-        const args = ['add', '--dev', moduleName];
+        const args = ['add', '--group', 'dev', moduleName];
         if (moduleName === 'black') {
             args.push('--allow-prereleases');
         }

diff --git a/src/test/common/installer/poetryInstaller.unit.test.ts b/src/test/common/installer/poetryInstaller.unit.test.ts
@@ -103,7 +103,7 @@ suite('Module Installer - Poetry', () => {
 
         const info = await poetryInstaller.getExecutionInfo('something', uri);
 
-        assert.deepEqual(info, { args: ['add', '--dev', 'something'], execPath: 'poetry path' });
+        assert.deepEqual(info, { args: ['add', '--group', 'dev', 'something'], execPath: 'poetry path' });
     });
     test('Get executable info when installing black', async () => {
         const uri = Uri.file(__dirname);
@@ -115,7 +115,7 @@ suite('Module Installer - Poetry', () => {
         const info = await poetryInstaller.getExecutionInfo('black', uri);
 
         assert.deepEqual(info, {
-            args: ['add', '--dev', 'black', '--allow-prereleases'],
+            args: ['add', '--group', 'dev', 'black', '--allow-prereleases'],
             execPath: 'poetry path',
         });
     });