From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 740FA45EFE; Sat, 21 Dec 2024 00:16:36 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0B3EC40156; Sat, 21 Dec 2024 00:16:36 +0100 (CET) Received: from mail-pj1-f49.google.com (mail-pj1-f49.google.com [209.85.216.49]) by mails.dpdk.org (Postfix) with ESMTP id A4B4E4003C for ; Sat, 21 Dec 2024 00:16:35 +0100 (CET) Received: by mail-pj1-f49.google.com with SMTP id 98e67ed59e1d1-2ef714374c0so1981554a91.0 for ; Fri, 20 Dec 2024 15:16:35 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; t=1734736594; x=1735341394; darn=dpdk.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=SOKA0bDHYuY4cM2si1/l8VXYtQ1ohKAuFCWyZrdeJLA=; b=NaY2x4AW7D2htyVs82CzXAmjJ/E0LXAShvT3eIDBrDjppGeVwawYafFCEaIj4tLylh XOGV0mtQzp8GFrknXD/KfZ01ak1gGqEqJwrAXLz0HPwynfjUNtGFHx0xGN5y4fIl97QQ xv85EkkBre/qG7yVvsJJETFbgLWQ5Y+AeLbyU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1734736594; x=1735341394; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=SOKA0bDHYuY4cM2si1/l8VXYtQ1ohKAuFCWyZrdeJLA=; b=aZBs672ryfUQPx0NmJ3nQmszboZUkG6Y+dfivpP04sgvgVuKu3uSGksx/R4JUGyISB NjJUdsDTn6PeWoqsK84wK5Joxdvat6CXXy+HuhtUj4e5PvhG6Z1GoIjo0C0j9aEuS0+C u4h3yWpLlKhxuFr+AFSkcL0lrLX3tPDzMyb6dlU8MDi9ELa1mc2J2gGripGfYlFQA9Ku 69asziOi9RCYTuIzr/AatO6x7hURJ+SPW6SrQ7yBcKQcPgLgIRmfOJ3F61NOL3rnPwXc I4k2obky3m7eP5ZUN0XoF61IROYSP5KgtmnTPL1y93iNTrlgu11DbY/2jQ4AtIIRa49l kKlQ== X-Gm-Message-State: AOJu0YwGIzj5rYFeTQTp70WutNengQA2d1t0NWTgz1lWUQA0j2tiJX3D EqUXvlY9ABQUadFOdO6E/KJmvfu37I2NvPREn+/YkumgxF+3l7V7saKKzl/+R/2bFnLwNZ3zB4c DmE/xLe9URJRzXDjDHIzd+l6up+6/36gG4D9kFUR/jnfcpeUQ X-Gm-Gg: ASbGnctnr3vCXFnvDuLLx5qm0qEkJoqw3W7ndT8KBQKxqCB+gfTbU7a79LN0BE34wzU 4c4+ZL0+m5vkaGkZkZbnk/0nVts/2caAS+WsO2LdRcEzjalIIAk1DNIRA6+Orf9CZiT2RapU= X-Google-Smtp-Source: AGHT+IGGU20+tPh/LZ1Tk74i+qci/3bleshtT3UuHQWi/Z/K6ugdGyR5Eexhod+sNM7+YXO18IWxp6dcUWc2+i+aEKY= X-Received: by 2002:a17:90b:2702:b0:2f4:4431:51d4 with SMTP id 98e67ed59e1d1-2f44431533fmr12826283a91.6.1734736594292; Fri, 20 Dec 2024 15:16:34 -0800 (PST) MIME-Version: 1.0 References: <20241210103253.3931003-1-luca.vizzarro@arm.com> <20241212140013.17548-1-luca.vizzarro@arm.com> In-Reply-To: <20241212140013.17548-1-luca.vizzarro@arm.com> From: Patrick Robb Date: Fri, 20 Dec 2024 18:14:06 -0500 Message-ID: Subject: Re: [PATCH v2 0/7] dts: add Ruff and docstring linting To: Luca Vizzarro Cc: dev@dpdk.org, Paul Szczepanek Content-Type: multipart/alternative; boundary="0000000000007b6ac60629bbd62d" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org --0000000000007b6ac60629bbd62d Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Series-reviewed-by: Patrick Robb Tested-by: Patrick Robb Paul I will merge to next-dts now instead of waiting until after Winter holidays if that is okay with you. On Thu, Dec 12, 2024 at 9:02=E2=80=AFAM Luca Vizzarro wrote: > v2: > - updated the doc page > > Luca Vizzarro (7): > dts: add Ruff as linter and formatter > dts: enable Ruff preview pydoclint rules > dts: resolve docstring linter errors > dts: apply Ruff formatting > dts: update dts-check-format to use Ruff > dts: remove old linters and formatters > dts: update linters in doc page > > devtools/dts-check-format.sh | 30 +-- > doc/guides/tools/dts.rst | 26 +-- > dts/framework/params/eal.py | 5 +- > dts/framework/remote_session/dpdk_shell.py | 1 - > dts/framework/remote_session/python_shell.py | 1 + > .../single_active_interactive_shell.py | 3 +- > dts/framework/runner.py | 14 +- > dts/framework/settings.py | 3 + > dts/framework/test_suite.py | 6 +- > dts/framework/testbed_model/capability.py | 13 +- > dts/framework/testbed_model/cpu.py | 21 +- > dts/framework/testbed_model/linux_session.py | 6 +- > dts/framework/testbed_model/node.py | 3 + > dts/framework/testbed_model/os_session.py | 3 +- > dts/framework/testbed_model/port.py | 1 - > dts/framework/testbed_model/posix_session.py | 16 +- > dts/framework/testbed_model/sut_node.py | 2 +- > dts/framework/testbed_model/topology.py | 6 + > .../traffic_generator/__init__.py | 3 + > .../testbed_model/traffic_generator/scapy.py | 7 +- > .../traffic_generator/traffic_generator.py | 3 +- > dts/framework/utils.py | 6 +- > dts/poetry.lock | 197 +++--------------- > dts/pyproject.toml | 40 ++-- > dts/tests/TestSuite_vlan.py | 22 +- > 25 files changed, 179 insertions(+), 259 deletions(-) > > -- > 2.43.0 > > --0000000000007b6ac60629bbd62d Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Series-reviewed-by: Patrick Robb <probb@iol.unh.edu>
Tested-by: Patrick Robb <<= a href=3D"mailto:probb@iol.unh.edu">probb@iol.unh.edu>
Paul I will merge to next-dts now instead of waiting until afte= r Winter holidays if that is okay with you.

On T= hu, Dec 12, 2024 at 9:02=E2=80=AFAM Luca Vizzarro <luca.vizzarro@arm.com> wrote:
v2:
- updated the doc page

Luca Vizzarro (7):
=C2=A0 dts: add Ruff as linter and formatter
=C2=A0 dts: enable Ruff preview pydoclint rules
=C2=A0 dts: resolve docstring linter errors
=C2=A0 dts: apply Ruff formatting
=C2=A0 dts: update dts-check-format to use Ruff
=C2=A0 dts: remove old linters and formatters
=C2=A0 dts: update linters in doc page

=C2=A0devtools/dts-check-format.sh=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0 =C2=A0 =C2=A0 |=C2=A0 30 +--
=C2=A0doc/guides/tools/dts.rst=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 |=C2=A0 26 +--
=C2=A0dts/framework/params/eal.py=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 =C2=A05 +-
=C2=A0dts/framework/remote_session/dpdk_shell.py=C2=A0 =C2=A0 |=C2=A0 =C2= =A01 -
=C2=A0dts/framework/remote_session/python_shell.py=C2=A0 |=C2=A0 =C2=A01 +<= br> =C2=A0.../single_active_interactive_shell.py=C2=A0 =C2=A0 =C2=A0 =C2=A0 |= =C2=A0 =C2=A03 +-
=C2=A0dts/framework/runner.py=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 14 +-
=C2=A0dts/framework/settings.py=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 =C2=A03 +
=C2=A0dts/framework/test_suite.py=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 =C2=A06 +-
=C2=A0dts/framework/testbed_model/capability.py=C2=A0 =C2=A0 =C2=A0|=C2=A0 = 13 +-
=C2=A0dts/framework/testbed_model/cpu.py=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 |=C2=A0 21 +-
=C2=A0dts/framework/testbed_model/linux_session.py=C2=A0 |=C2=A0 =C2=A06 +-=
=C2=A0dts/framework/testbed_model/node.py=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0|=C2=A0 =C2=A03 +
=C2=A0dts/framework/testbed_model/os_session.py=C2=A0 =C2=A0 =C2=A0|=C2=A0 = =C2=A03 +-
=C2=A0dts/framework/testbed_model/port.py=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0|=C2=A0 =C2=A01 -
=C2=A0dts/framework/testbed_model/posix_session.py=C2=A0 |=C2=A0 16 +-
=C2=A0dts/framework/testbed_model/sut_node.py=C2=A0 =C2=A0 =C2=A0 =C2=A0|= =C2=A0 =C2=A02 +-
=C2=A0dts/framework/testbed_model/topology.py=C2=A0 =C2=A0 =C2=A0 =C2=A0|= =C2=A0 =C2=A06 +
=C2=A0.../traffic_generator/__init__.py=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0|=C2=A0 =C2=A03 +
=C2=A0.../testbed_model/traffic_generator/scapy.py=C2=A0 |=C2=A0 =C2=A07 +-=
=C2=A0.../traffic_generator/traffic_generator.py=C2=A0 =C2=A0 |=C2=A0 =C2= =A03 +-
=C2=A0dts/framework/utils.py=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 |=C2=A0 =C2=A06 +-
=C2=A0dts/poetry.lock=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 197 +++-------= --------
=C2=A0dts/pyproject.toml=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 |=C2=A0 40 ++--
=C2=A0dts/tests/TestSuite_vlan.py=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 22 +-
=C2=A025 files changed, 179 insertions(+), 259 deletions(-)

--
2.43.0

--0000000000007b6ac60629bbd62d--