Restarting task should read the updated content#251755
Conversation
| UnknownError | ||
| } | ||
|
|
||
| export class VerifiedTask { |
There was a problem hiding this comment.
Had to move this here for code layer!
|
Proof of testing both restarting on going task and re-running last task: restarting: rerunning last task: Screen.Recording.2025-06-18.at.12.31.14.PM.mov |
|
@anthonykim1 thanks for working on this! Can you test this on a longrunning task for which you get the notification "the task is already active" vs a single run task like the one in your gif? |
|
@meganrogge Thanks for reviewing! Yeah, actually I'm not sure why that notification didn't show up in the first gif (the one that prints for 30 second). Edit: Oh, wait I'm finally able to get the notification, but this only shows up sometimes when I re-run, not when I restart from command palette:
Just found a bug where re-starting non-changed compile task leads to error, will investigate and fix! |
|
Got the bug resolved! Here are some testing GIFs @meganrogge: Screen.Recording.2025-06-19.at.11.51.00.PM.movScreen.Recording.2025-06-19.at.11.53.44.PM.movScreen.Recording.2025-06-19.at.11.54.42.PM.mov |
meganrogge
left a comment
There was a problem hiding this comment.
LGTM, thanks so much Anthony!
Resolves: #169267
Fixing two things in this PR: