Skip to content

Commit

Permalink
Fix task delete debt
Browse files Browse the repository at this point in the history
Part of #96022
  • Loading branch information
alexr00 committed Apr 24, 2020
1 parent 03be1bb commit 2c76aee
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/vs/workbench/contrib/tasks/browser/taskQuickPick.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

import * as nls from 'vs/nls';
import * as Objects from 'vs/base/common/objects';
import { Task, ContributedTask, CustomTask, ConfiguringTask, TaskSorter } from 'vs/workbench/contrib/tasks/common/tasks';
import { Task, ContributedTask, CustomTask, ConfiguringTask, TaskSorter, KeyedTaskIdentifier } from 'vs/workbench/contrib/tasks/common/tasks';
import { IWorkspace, IWorkspaceFolder } from 'vs/platform/workspace/common/workspace';
import * as Types from 'vs/base/common/types';
import { ITaskService, WorkspaceFolderTaskResult } from 'vs/workbench/contrib/tasks/common/taskService';
Expand Down Expand Up @@ -52,7 +52,7 @@ export class TaskQuickPick extends Disposable {
}
if (ConfiguringTask.is(task)) {
let label: string = task.configures.type;
const configures = Objects.deepClone(task.configures);
const configures: Partial<KeyedTaskIdentifier> = Objects.deepClone(task.configures);
delete configures['_key'];
delete configures['type'];
Object.keys(configures).forEach(key => label += `: ${configures[key]}`);
Expand Down
2 changes: 1 addition & 1 deletion src/vs/workbench/contrib/tasks/common/taskConfiguration.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1197,7 +1197,7 @@ namespace ProblemMatcherConverter {
if (global) {
return Objects.deepClone(global);
}
let localProblemMatcher = context.namedProblemMatchers[variableName];
let localProblemMatcher: ProblemMatcher & Partial<NamedProblemMatcher> = context.namedProblemMatchers[variableName];
if (localProblemMatcher) {
localProblemMatcher = Objects.deepClone(localProblemMatcher);
// remove the name
Expand Down

0 comments on commit 2c76aee

Please sign in to comment.