On Thu, Sep 4, 2025 at 11:16 AM Luca Vizzarro <luca.vizzarro@arm.com> wrote:

> On Wed, Sep 03, 2025 at 02:04:12PM +0000, Andrew Bailey wrote:
> > diff --git a/dts/framework/remote_session/testpmd_shell.py
> b/dts/framework/remote_session/testpmd_shell.py
> > index ad8cb273dc..4d9caceb37 100644
> > --- a/dts/framework/remote_session/testpmd_shell.py
> > +++ b/dts/framework/remote_session/testpmd_shell.py
> > @@ -19,7 +19,7 @@
> >  import time
> >  from collections.abc import Callable, MutableSet
> >  from dataclasses import dataclass, field
> > -from enum import Flag, auto
> > +from enum import Enum, Flag, auto
> >  from os import environ
> >  from pathlib import PurePath
> >  from typing import TYPE_CHECKING, Any, ClassVar, Concatenate, Literal,
> ParamSpec, Tuple, TypeAlias
> > @@ -344,6 +344,13 @@ def make_parser(cls) -> ParserFn:
> >          )
> >
> >
> > +class RxTxArgFlag(Enum):
>
> this could be StrEnum given the values used. Moreover, is it a
> flag? Looks like a regular enum to me.
>

I was wondering the same but no, by my understanding it's currently an enum
or StrEnum and I can't see us adding additional attributes in the future so
that won't be changing.