On Thu, Sep 4, 2025 at 11:16 AM Luca Vizzarro <luca.vizza...@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.
