Skip to content

Commit

Permalink
Ensure new Proposed API is always available (#14128)
Browse files Browse the repository at this point in the history
  • Loading branch information
DonJayamanne authored Aug 16, 2023
1 parent 0833281 commit 341812a
Showing 1 changed file with 2 additions and 7 deletions.
9 changes: 2 additions & 7 deletions src/kernels/jupyter/finder/remoteKernelFinderController.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

import { injectable, inject } from 'inversify';
import { IKernelFinder, IKernelProvider } from '../../types';
import { IDisposableRegistry, IExtensionContext, IFeaturesManager } from '../../../platform/common/types';
import { IDisposableRegistry, IExtensionContext } from '../../../platform/common/types';
import {
IOldJupyterSessionManagerFactory,
IJupyterServerUriStorage,
Expand Down Expand Up @@ -49,9 +49,7 @@ export class RemoteKernelFinderController implements IRemoteKernelFinderControll
@inject(IJupyterUriProviderRegistration)
private readonly jupyterPickerRegistration: IJupyterUriProviderRegistration,
@inject(IJupyterServerProviderRegistry)
private readonly jupyterServerProviderRegistry: IJupyterServerProviderRegistry,
@inject(IFeaturesManager)
private readonly features: IFeaturesManager
private readonly jupyterServerProviderRegistry: IJupyterServerProviderRegistry
) {}
private readonly handledProviders = new WeakSet<IJupyterUriProvider>();
activate() {
Expand Down Expand Up @@ -92,9 +90,6 @@ export class RemoteKernelFinderController implements IRemoteKernelFinderControll
private mappedServers = new Set<string>();
@swallowExceptions('Handle Jupyter Provider Changes')
private async handleProviderChanges() {
if (!this.features.features.enableProposedJupyterServerProviderApi) {
return;
}
if (!this.serverUriStorage.all.length) {
// We do not have any of the previously used servers, or the data has not yet loaded.
return;
Expand Down

0 comments on commit 341812a

Please sign in to comment.