DPDK patches and discussions
 help / color / mirror / Atom feed
From: Patrick Robb <probb@iol.unh.edu>
To: Luca Vizzarro <luca.vizzarro@arm.com>
Cc: dev@dpdk.org, "Juraj Linkeš" <juraj.linkes@pantheon.tech>,
	"Jeremy Spewock" <jspewock@iol.unh.edu>
Subject: Re: [PATCH v2 0/2] dts: update mypy and clean up
Date: Fri, 24 May 2024 12:17:48 -0400	[thread overview]
Message-ID: <CAJvnSUAi4=+RWa1XZtL5gqG-O73hAwQV35s3a2LgwEWd1jFWrA@mail.gmail.com> (raw)
In-Reply-To: <20240514113446.1948050-1-luca.vizzarro@arm.com>

Series-reviewed-by: Patrick Robb <probb@iol.unh.edu>

As a reminder, once this is merged, UNH will add a CI job which runs
mypy and reports its results to PW (only runs for patches with changes
in dpdk/dts).

Luca, I assume what you had in mind was we will run
dts-check-format.sh, so including isort and pylama, as opposed to just
mypy? Thanks.

On Tue, May 14, 2024 at 7:34 AM Luca Vizzarro <luca.vizzarro@arm.com> wrote:
>
> v2:
> - apply fix to error formatting bug in
>   framework/testbed_model/traffic_generator/__init__.py
>
> Luca Vizzarro (2):
>   dts: update mypy static checker
>   dts: clean up config types
>
>  dts/framework/config/__init__.py              | 49 ++++++-----
>  dts/framework/logger.py                       |  4 +-
>  .../interactive_remote_session.py             |  4 +-
>  .../remote_session/interactive_shell.py       |  2 +-
>  dts/framework/remote_session/ssh_session.py   |  6 +-
>  dts/framework/runner.py                       | 14 +--
>  dts/framework/test_result.py                  |  4 +-
>  dts/framework/test_suite.py                   |  6 +-
>  dts/framework/testbed_model/tg_node.py        |  2 +-
>  .../traffic_generator/__init__.py             | 10 +--
>  .../capturing_traffic_generator.py            |  4 +-
>  .../testbed_model/traffic_generator/scapy.py  |  6 +-
>  .../traffic_generator/traffic_generator.py    |  2 +-
>  dts/framework/utils.py                        |  2 +-
>  dts/poetry.lock                               | 86 +++++++++++--------
>  dts/pyproject.toml                            |  3 +-
>  dts/tests/TestSuite_os_udp.py                 |  4 +-
>  dts/tests/TestSuite_pmd_buffer_scatter.py     |  8 +-
>  18 files changed, 118 insertions(+), 98 deletions(-)
>
> --
> 2.34.1
>

  parent reply	other threads:[~2024-05-24 16:18 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-09 10:57 [PATCH " Luca Vizzarro
2024-05-09 10:57 ` [PATCH 1/2] dts: update mypy static checker Luca Vizzarro
2024-05-13 16:02   ` Juraj Linkeš
2024-05-14 11:30     ` Luca Vizzarro
2024-05-29 11:22       ` Juraj Linkeš
2024-05-09 10:57 ` [PATCH 2/2] dts: clean up config types Luca Vizzarro
2024-05-13 16:12   ` Juraj Linkeš
2024-05-14 11:30     ` Luca Vizzarro
2024-05-29 11:41   ` Juraj Linkeš
2024-05-14 11:34 ` [PATCH v2 0/2] dts: update mypy and clean up Luca Vizzarro
2024-05-14 11:34   ` [PATCH v2 1/2] dts: update mypy static checker Luca Vizzarro
2024-05-24 14:51     ` Patrick Robb
2024-05-28 12:00       ` Luca Vizzarro
2024-05-29 10:58         ` Juraj Linkeš
2024-05-29 11:39     ` Juraj Linkeš
2024-05-29 15:20       ` Luca Vizzarro
2024-05-14 11:34   ` [PATCH v2 2/2] dts: clean up config types Luca Vizzarro
2024-05-24 16:13     ` Patrick Robb
2024-05-20 15:37   ` [PATCH v2 0/2] dts: update mypy and clean up Nicholas Pratte
2024-05-21 14:59     ` Luca Vizzarro
2024-05-24 13:47       ` Nicholas Pratte
2024-05-24 16:17   ` Patrick Robb [this message]
2024-05-28 12:05     ` Luca Vizzarro
2024-05-29 10:44       ` Juraj Linkeš
2024-05-30 15:19 ` [PATCH v3 " Luca Vizzarro
2024-05-30 15:19   ` [PATCH v3 1/2] dts: update mypy static checker Luca Vizzarro
2024-05-30 20:23     ` Jeremy Spewock
2024-05-31  9:29     ` Juraj Linkeš
2024-06-17 15:28     ` Nicholas Pratte
2024-05-30 15:19   ` [PATCH v3 2/2] dts: clean up config types Luca Vizzarro
2024-05-30 20:23     ` Jeremy Spewock
2024-05-31  9:29     ` Juraj Linkeš
2024-06-17 15:30     ` Nicholas Pratte
2024-06-19 23:02   ` [PATCH v3 0/2] dts: update mypy and clean up Thomas Monjalon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAJvnSUAi4=+RWa1XZtL5gqG-O73hAwQV35s3a2LgwEWd1jFWrA@mail.gmail.com' \
    --to=probb@iol.unh.edu \
    --cc=dev@dpdk.org \
    --cc=jspewock@iol.unh.edu \
    --cc=juraj.linkes@pantheon.tech \
    --cc=luca.vizzarro@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).