Test failure shouldn't cause other tests to cancel execution

[justafish]

• When a parallel task fails allow the other parallel tasks to finish before exiting with failure go-task/task#1246
• https://github.com/go-task/task/blob/c4dce8f506814d7af750792f53d4fa0c863b4470/task.go#L291

[elvism-lullabot]

@justafish @deviantintegral does this Issue have to do with the cancel-in-progress configuration?

[justafish]

@elvism-lullabot no this is an issue with task. Say for example you wanted to run all your project's tests at once, a failure in one of the types of tests would halt execution in the others e.g. phpcs fails and kills all the other processes like PHPUnit

[elvism-lullabot]

@justafish This is ready for review.

[mrdavidburns]

This would ideally be fixed upstream in go-task/task#1246

[elvism-lullabot]

I’ve created a PR for the Task issue: go-task/task#2307.
Next, I’ll look into how I can properly test it within this repository.

[deviantintegral]

@rabbitlair could you test and review the upstream PR, and leave a comment there with anything you find?