maxmustermann/vscode
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Remove code that duplicates Array.join from runAutomaticTasks.ts

Browse source 
Fixes #64602
This commit is contained in:
Alex Ross 2018-12-10 12:52:57 +01:00
parent 5670416e4a
commit b97740c4e6
1 changed files with 1 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/vs/workbench/parts/tasks/electron-browser/runAutomaticTasks.ts
View file

@ -89,14 +89,8 @@ export class RunAutomaticTasks extends Disposable implements IWorkbenchContribut
}
private showPrompt(taskNames: Array<string>): Promise<boolean> {
// We will only show the prompt if their are automatic tasks, so taskNames is at least of length 1.
let taskNamesPrint: string = '(' + taskNames[0];
for (let i = 1; i < taskNames.length; i++) {
taskNamesPrint += ', ' + taskNames[i];
}
taskNamesPrint += ')';
return new Promise<boolean>(resolve => {
this.notificationService.prompt(Severity.Info, nls.localize('tasks.run.allowAutomatic', "This folder has tasks {0} defined in \'tasks.json\' that run automatically when you open this folder. Do you allow automatic tasks to run when you open this folder?", taskNamesPrint),
this.notificationService.prompt(Severity.Info, nls.localize('tasks.run.allowAutomatic', "This folder has tasks ({0}) defined in \'tasks.json\' that run automatically when you open this folder. Do you allow automatic tasks to run when you open this folder?", taskNames.join(', ')),
[{
label: nls.localize('allow', "Allow"),
run: () => {