Re: [PATCH v2 08/15] iotests/297: Change run_linter() to raise an exception on failure

2021-10-26 Thread John Snow
On Tue, Oct 26, 2021 at 6:10 AM Hanna Reitz wrote: > On 19.10.21 16:49, John Snow wrote: > > Instead of using a process return code as the python function return > > value (or just not returning anything at all), allow run_linter() to > > raise an exception instead. > > > > The responsibility

Re: [PATCH v2 08/15] iotests/297: Change run_linter() to raise an exception on failure

2021-10-26 Thread Hanna Reitz
On 19.10.21 16:49, John Snow wrote: Instead of using a process return code as the python function return value (or just not returning anything at all), allow run_linter() to raise an exception instead. The responsibility for printing output on error shifts from the function itself to the

[PATCH v2 08/15] iotests/297: Change run_linter() to raise an exception on failure

2021-10-19 Thread John Snow
Instead of using a process return code as the python function return value (or just not returning anything at all), allow run_linter() to raise an exception instead. The responsibility for printing output on error shifts from the function itself to the caller, who will know best how to