fix(core): run post-task lifecycle for run-command parallel exections #33572
+5
−5
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Current Behavior
When using a plugin with
postTasksExecution, this is never called in some complex scenarios with continuous tasks.For instance, we have a task
devwhich runs the server, and then depends onwatch-deps(both continuous). In these scenarios, we never get stats output when the NX run is killed. However, if we remove the watch-deps it seems to work.Expected Behavior
All invocations of NX which run tasks should ultimately output task data.
NOTE: This changes the output in non-TUI mode to include the second command which was lost before:
Related Issue(s)
This is related to #33562, but potentially upstream of that as well. I've tried to test the changes together and ensure that not only do we get invocations, but we see the continuous tasks as well
Closes #33586