On Wed, Jul 8, 2026 at 9:05 PM Patrick Robb <[email protected]> wrote:
> Actually, please see the AI code review of this patch. Specifically, this > comment looks correct to me. What are your thoughts? > > **File:** `dts/framework/testbed_model/linux_session.py`, line 228 > > The code checks `lshw_result.return_code != 0 and lshw_result.stdout == > ""` but the logic is incorrect. If the command fails, you should raise an > error. The `and` should likely be `or`: > This is definitely a valid issue, I will send out a patch fixing this shortly. > > Please read through the other suggestions too. There is some fluff but > also some good advice. > For sure, I will include changes from those suggestions in the subsequent patch as well.
