From c6e228bba0a0ff5b2e7c1074a23f53fc37923f02 Mon Sep 17 00:00:00 2001 From: Brian Haley Date: Tue, 4 Nov 2025 14:51:01 +0100 Subject: [PATCH] Update to pylint 3.3.0 Start using pylint 3.3.0 to fix a pep8 job failure. Fixed R1737 (use-yield-from) warning. Added too-many-positional-arguments to pylint ignore list. Related-bug: #2130632 Change-Id: Ie043024a458cfc05dc95b7a4cc52ac433ba73df9 Signed-off-by: Brian Haley --- .pylintrc | 1 + ovsdbapp/backend/ovs_idl/idlutils.py | 3 +-- tox.ini | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.pylintrc b/.pylintrc index 79b6e378..27fb3cfb 100644 --- a/.pylintrc +++ b/.pylintrc @@ -89,6 +89,7 @@ disable= too-many-locals, too-many-nested-blocks, too-many-public-methods, + too-many-positional-arguments, too-many-return-statements, too-many-statements, consider-using-set-comprehension, diff --git a/ovsdbapp/backend/ovs_idl/idlutils.py b/ovsdbapp/backend/ovs_idl/idlutils.py index 060d4fcb..a4b467b1 100644 --- a/ovsdbapp/backend/ovs_idl/idlutils.py +++ b/ovsdbapp/backend/ovs_idl/idlutils.py @@ -281,8 +281,7 @@ def circular(*items): if not items: return while True: - for x in items: - yield x + yield from items def merge_intersection(*sorted_gens): diff --git a/tox.ini b/tox.ini index 7502e9d0..a92009af 100644 --- a/tox.ini +++ b/tox.ini @@ -26,7 +26,7 @@ commands = flake8 bash {toxinidir}/tools/coding-checks.sh --all '{posargs}' deps = {[testenv]deps} - pylint==2.17.4 # GPLv2 + pylint==3.3.0 # GPLv2 allowlist_externals = bash [testenv:venv]