Re: [PATCH v2 1/2] dts: update mypy static checker

2024-05-29 Thread Luca Vizzarro
On 29/05/2024 12:39, Juraj Linkeš wrote: diff --git a/dts/poetry.lock b/dts/poetry.lock index a734fa71f0..df9cecb7e0 100644 --- a/dts/poetry.lock +++ b/dts/poetry.lock @@ -1,4 +1,4 @@ -# This file is automatically @generated by Poetry 1.5.1 and should not be changed by hand. +# This file is auto

Re: [PATCH v2 1/2] dts: update mypy static checker

2024-05-29 Thread Juraj Linkeš
> diff --git a/dts/poetry.lock b/dts/poetry.lock > index a734fa71f0..df9cecb7e0 100644 > --- a/dts/poetry.lock > +++ b/dts/poetry.lock > @@ -1,4 +1,4 @@ > -# This file is automatically @generated by Poetry 1.5.1 and should not be > changed by hand. > +# This file is automatically @generated by Poe

Re: [PATCH v2 1/2] dts: update mypy static checker

2024-05-29 Thread Juraj Linkeš
On Tue, May 28, 2024 at 2:00 PM Luca Vizzarro wrote: > > On 24/05/2024 15:51, Patrick Robb wrote: > > I think this is fine. Another option would be to move sut_nodes and > > tg_nodes up as DTSRunner attributes. This is a good suggestion. Everything in DTSRunner is tied to a particular execution/b

Re: [PATCH v2 1/2] dts: update mypy static checker

2024-05-28 Thread Luca Vizzarro
On 24/05/2024 15:51, Patrick Robb wrote: I think this is fine. Another option would be to move sut_nodes and tg_nodes up as DTSRunner attributes. I like preserving the type hint, but it also might just be cluttering the code to do this... up to you. I made the change because of a complaint comi

Re: [PATCH v2 1/2] dts: update mypy static checker

2024-05-24 Thread Patrick Robb
On Tue, May 14, 2024 at 7:34 AM Luca Vizzarro wrote: > diff --git a/dts/framework/runner.py b/dts/framework/runner.py > index db8e3ba96b..d74f1871db 100644 > --- a/dts/framework/runner.py > +++ b/dts/framework/runner.py > @@ -23,7 +23,7 @@ > import re > import sys > from pathlib import Path > -

[PATCH v2 1/2] dts: update mypy static checker

2024-05-14 Thread Luca Vizzarro
Update the mypy static checker to the latest version and fix all the reported errors. Bugzilla ID: 1433 Signed-off-by: Luca Vizzarro Reviewed-by: Paul Szczepanek --- dts/framework/config/__init__.py | 2 +- dts/framework/logger.py | 4 +- .../interactive_re