This series cleans up all pylint warnings in ovs-dpctl.py, bringing the score from 7.62/10 to 10.00/10.
This series applies on top of: [PATCH net-next v10 1/2] selftests: openvswitch: add vlan() and encap() flow string parsing https://lore.kernel.org/netdev/[email protected]/ Patch 1 converts 86 %-format strings to f-strings. Patch 2 fixes misc warnings (unused import, bare except, unused variables, redundant expressions). Patch 3 renames classes to PascalCase and variables to snake_case. Patch 4 adds one-line docstrings to all definitions. Patch 5 suppresses complexity warnings from pyroute2 constraints. Tested with vng on x86_64, all OVS selftests pass. Minxi Hou (5): selftests: openvswitch: convert %-formatting to f-strings selftests: openvswitch: fix misc pylint warnings in ovs-dpctl.py selftests: openvswitch: rename classes and variables in ovs-dpctl.py selftests: openvswitch: add missing docstrings in ovs-dpctl.py selftests: openvswitch: suppress pylint complexity warnings .../selftests/net/openvswitch/ovs-dpctl.py | 794 ++++++++++-------- 1 file changed, 428 insertions(+), 366 deletions(-) -- 2.53.0
