|SUCCESS| dts: fix Sphinx parsing of a docstring
dpdklab at iol.unh.edu
dpdklab at iol.unh.edu
Wed Apr 1 05:13:42 CEST 2026
Test-Label: ai-code-review
Test-Status: SUCCESS
https://dpdk.org/patch/162800
_AI Code Review_
=== Patch Review: series.patch (via Claude) ===
Review date: 2026-04-01
This patch makes a minor change to a docstring in a DTS Python file. Let me review it carefully.
The patch splits a single-line docstring into two lines as a workaround for a Sphinx parsing issue where "ports" is incorrectly treated as a reference.
After reviewing the patch, I have found no regressions. The change is purely cosmetic - it breaks a single-line comment into two lines while preserving all content. The formatting follows Python conventions for multi-line comments, and the change does not affect code functionality or introduce any coding standard violations.
---
git sha: (not provided in patch)
Author: Thomas Monjalon <thomas at monjalon.net>
dts: fix Sphinx parsing of a docstring
A brief summary: This patch splits a docstring into two lines to work
around a Sphinx 9.1.0 parsing issue where the word "ports" is incorrectly
treated as a reference.
> diff --git a/dts/api/testpmd/config.py b/dts/api/testpmd/config.py
> index e71a3e1ef0..1e59bccd08 100644
> --- a/dts/api/testpmd/config.py
> +++ b/dts/api/testpmd/config.py
[ ... ]
> @@ -30,7 +30,8 @@
> class PortTopology(StrEnum):
> """Enum representing the port topology."""
>
> - #: In paired mode, the forwarding is between pairs of ports, e.g.: (0,1), (2,3), (4,5).
> + #: In paired mode, the forwarding is between pairs of ports, e.g.:
> + #: (0,1), (2,3), (4,5).
> paired = auto()
>
> #: In chained mode, the forwarding is to the next available port in the port mask, e.g.:
No regressions found.
More information about the test-report
mailing list