fix for "Cannot Create keybinding to restart a specific Task" #36474
fix for "Cannot Create keybinding to restart a specific Task" #36474vemoo wants to merge 9 commits intomicrosoft:masterfrom
Conversation
|
I fixed some typos in some translation keys in this commit becase at the time I could't find any translations in the repository. But now I see there are translations, should I revert that change? or update the keys in the translations also? |
|
There is no need to revert nor to update the translation. This happens automatically when we pull the next time from our translation database. |
|
After using vscode for a while I realised this pull request introduces some changes when stopping and restarting tasks. Right now when terminating or restarting a task, the following happens:
In this pull request I unified the code so both stop or terminate the task without asking the user if there's only one running. Also, if an argument is passed but no task is found, should there be some kind of notification for the user? |
|
In tasks 2.0 we always prompt on terminate even if only one task is running. Since task 2.0.0 can run multiple task in parallel we need a dialog for n >= 2 and to make things consistent we always show the dialog. However if a keyboard short cut is bound to terminate a specific task no picker should be shown. If an argument is passed but the task is not found I would show a picker with the currently running tasks. This is consistent with a short cut to executing a task. |
|
Is there anything else I can do to help this pull request get merged? |
|
@vemoo actually I was not sure if your PR addresses these two issues:
|
|
Yes, with this PR, when terminating a task with an argument
|
… matching task is found
…is passed and the task is found, or when we are in the old task system
|
I've rebased onto current upstream master, and fixed the conflict. |
|
The fix for the arg is OK, however there is #47853 which adds a more generic way to skip zero task prompts. Will look into merging the two PRs. |
|
A lot has changed here especially the fact to be able to reference a task via a task identifier. So I merged the PR by hand and modified it to:
|
Issue: #35910.
Also fixes a couple of related issues in src/vs/workbench/parts/tasks/node/processTaskSystem.ts task termination.