vezel-dev / cathode

A terminal-centric replacement for the .NET console APIs.
https://docs.vezel.dev/cathode
BSD Zero Clause License
91 stars 7 forks source link

Kill entire `ChildProcess` tree on cancellation #85

Closed alexrp closed 2 years ago

alexrp commented 2 years ago

https://github.com/vezel-dev/cathode/blob/3464a162077b3c1313fd86a3c8c9f73bae2c87a4/src/core/Processes/ChildProcess.cs#L128-L141

alexrp commented 2 years ago

Decided not to make it configurable as I think this is a very reasonable default. Am open to adding a configuration knob if a real use case turns up.