Skip to content

Commit

Permalink
Remove old prompt from tasks service
Browse files Browse the repository at this point in the history
Part of #119463
  • Loading branch information
alexr00 committed Apr 22, 2021
1 parent 00e8e64 commit 29ade59
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 23 deletions.
22 changes: 2 additions & 20 deletions src/vs/workbench/contrib/tasks/browser/abstractTaskService.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ import { IStorageService, StorageScope, StorageTarget } from 'vs/platform/storag
import { IProgressService, IProgressOptions, ProgressLocation } from 'vs/platform/progress/common/progress';

import { IOpenerService } from 'vs/platform/opener/common/opener';
import { IHostService } from 'vs/workbench/services/host/browser/host';
import { INotificationService } from 'vs/platform/notification/common/notification';
import { IDialogService } from 'vs/platform/dialogs/common/dialogs';

Expand Down Expand Up @@ -248,7 +247,6 @@ export abstract class AbstractTaskService extends Disposable implements ITaskSer
@IStorageService private readonly storageService: IStorageService,
@IProgressService private readonly progressService: IProgressService,
@IOpenerService private readonly openerService: IOpenerService,
@IHostService private readonly _hostService: IHostService,
@IDialogService protected readonly dialogService: IDialogService,
@INotificationService private readonly notificationService: INotificationService,
@IContextKeyService protected readonly contextKeyService: IContextKeyService,
Expand Down Expand Up @@ -276,24 +274,8 @@ export abstract class AbstractTaskService extends Disposable implements ITaskSer
}
let folderSetup = this.computeWorkspaceFolderSetup();
if (this.executionEngine !== folderSetup[2]) {
if (this._taskSystem && this._taskSystem.getActiveTasks().length > 0) {
this.notificationService.prompt(
Severity.Info,
nls.localize(
'TaskSystem.noHotSwap',
'Changing the task execution engine with an active task running requires to reload the Window'
),
[{
label: nls.localize('reloadWindow', "Reload Window"),
run: () => this._hostService.reload()
}],
{ sticky: true }
);
return;
} else {
this.disposeTaskSystemListeners();
this._taskSystem = undefined;
}
this.disposeTaskSystemListeners();
this._taskSystem = undefined;
}
this.updateSetup(folderSetup);
this.updateWorkspaceTasks();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@ import { IConfigurationResolverService } from 'vs/workbench/services/configurati
import { IEditorService } from 'vs/workbench/services/editor/common/editorService';
import { IWorkbenchEnvironmentService } from 'vs/workbench/services/environment/common/environmentService';
import { IExtensionService } from 'vs/workbench/services/extensions/common/extensions';
import { IHostService } from 'vs/workbench/services/host/browser/host';
import { ILifecycleService } from 'vs/workbench/services/lifecycle/common/lifecycle';
import { IPanelService } from 'vs/workbench/services/panel/common/panelService';
import { IPathService } from 'vs/workbench/services/path/common/pathService';
Expand Down Expand Up @@ -72,7 +71,6 @@ export class TaskService extends AbstractTaskService {
@IStorageService storageService: IStorageService,
@IProgressService progressService: IProgressService,
@IOpenerService openerService: IOpenerService,
@IHostService _hostService: IHostService,
@IDialogService dialogService: IDialogService,
@INotificationService notificationService: INotificationService,
@IContextKeyService contextKeyService: IContextKeyService,
Expand Down Expand Up @@ -103,7 +101,6 @@ export class TaskService extends AbstractTaskService {
storageService,
progressService,
openerService,
_hostService,
dialogService,
notificationService,
contextKeyService,
Expand Down

0 comments on commit 29ade59

Please sign in to comment.