Skip to content

Commit 1d8d49f

Browse files
edvilmeCopilot
andcommitted
Remove setPackages from PackageManager interface
Remove setPackages from PackageManager interface in src/api.ts, api/src/main.ts, and examples/sample1/src/api.ts, and its wrapper implementation in internal.api.ts. No managers implemented this method. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent af77b33 commit 1d8d49f

4 files changed

Lines changed: 0 additions & 44 deletions

File tree

api/src/main.ts

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -689,18 +689,6 @@ export interface PackageManager {
689689
*/
690690
getPackages(environment: PythonEnvironment, options?: GetPackagesOptions): Promise<Package[] | undefined>;
691691

692-
/**
693-
* Updates the cached packages for the specified environment and fires a change event.
694-
* @param environment - The Python environment whose packages changed.
695-
* @param packages - The new list of packages.
696-
* @param changes - The list of changes describing what was added or removed.
697-
*/
698-
setPackages(
699-
environment: PythonEnvironment,
700-
packages: Package[],
701-
changes: { kind: PackageChangeKind; pkg: Package }[],
702-
): void;
703-
704692
/**
705693
* Event that is fired when packages change.
706694
*/

examples/sample1/src/api.ts

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -616,18 +616,6 @@ export interface PackageManager {
616616
*/
617617
getPackages(environment: PythonEnvironment, options?: GetPackagesOptions): Promise<Package[] | undefined>;
618618

619-
/**
620-
* Updates the cached packages for the specified environment and fires a change event.
621-
* @param environment - The Python environment whose packages changed.
622-
* @param packages - The new list of packages.
623-
* @param changes - The list of changes describing what was added or removed.
624-
*/
625-
setPackages(
626-
environment: PythonEnvironment,
627-
packages: Package[],
628-
changes: { kind: PackageChangeKind; pkg: Package }[],
629-
): void;
630-
631619
/**
632620
* Event that is fired when packages change.
633621
*/

src/api.ts

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -684,18 +684,6 @@ export interface PackageManager {
684684
*/
685685
getPackages(environment: PythonEnvironment, options?: GetPackagesOptions): Promise<Package[] | undefined>;
686686

687-
/**
688-
* Updates the cached packages for the specified environment and fires a change event.
689-
* @param environment - The Python environment whose packages changed.
690-
* @param packages - The new list of packages.
691-
* @param changes - The list of changes describing what was added or removed.
692-
*/
693-
setPackages(
694-
environment: PythonEnvironment,
695-
packages: Package[],
696-
changes: { kind: PackageChangeKind; pkg: Package }[],
697-
): void;
698-
699687
/**
700688
* Event that is fired when packages change.
701689
*/

src/internal.api.ts

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -373,14 +373,6 @@ export class InternalPackageManager implements PackageManager {
373373
return this.manager.getPackages(environment, options);
374374
}
375375

376-
setPackages(
377-
environment: PythonEnvironment,
378-
packages: Package[],
379-
changes: { kind: PackageChangeKind; pkg: Package }[],
380-
): void {
381-
this.manager.setPackages(environment, packages, changes);
382-
}
383-
384376
onDidChangePackages(handler: (e: DidChangePackagesEventArgs) => void): Disposable {
385377
return this.manager.onDidChangePackages ? this.manager.onDidChangePackages(handler) : new Disposable(() => {});
386378
}

0 commit comments

Comments
 (0)