Merge pull request syslog-ng#5123 from HofiOne/fix-pylint-error

HofiOne · web-flow · commit a49d3832c292 · 2024-09-20T15:50:06.000+02:00
python venv: disable `too-many-positional-arguments` as well for pylint
diff --git a/modules/python-modules/Makefile.am b/modules/python-modules/Makefile.am
@@ -127,7 +127,7 @@ pymodules-unit: python-venv
 pymodules-checks: pymodules-unit pymodules-pycodestyle pymodules-pylint
 
 pymodules-pycodestyle: python-venv
-	$(PYTHON_VENV) -m pycodestyle --ignore=E501,E121,E123,E126,E226,E24,E704,W503,W504 $(PYMODULES_SRCDIR)/$(PYMODULES_ROOT_MODULE)
+	$(PYTHON_VENV) -m pycodestyle --ignore=E501,E121,E123,E126,E226,E24,E704,W503,W504,R0917 $(PYMODULES_SRCDIR)/$(PYMODULES_ROOT_MODULE)
 
 pymodules-pylint: python-venv
 	$(PYTHON_VENV) -m pylint -r n --rcfile=$(PYMODULES_SRCDIR)/pylintrc $(PYMODULES_SRCDIR)/$(PYMODULES_ROOT_MODULE)
diff --git a/modules/python-modules/syslogng/debuggercli/tests/test_langcompleter.py b/modules/python-modules/syslogng/debuggercli/tests/test_langcompleter.py
@@ -57,7 +57,7 @@ class TestLangCompleter(CompleterTestCase):
         'PARTIAL_TOKEN': ChoiceCompleter(("tokenP-a", "tokenP-b"), prefix='@', suffix='')
     }
 
-    # pylint: disable=arguments-differ,too-many-arguments
+    # pylint: disable=arguments-differ,too-many-arguments,too-many-positional-arguments
     def _construct_completer(self, expected_token=None, expected_tokens=None,
                              replaced_token=None, replaced_token_pos=-1,
                              completers=None, prefix="<!--"):

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ class TestLangCompleter(CompleterTestCase):`
`57`	`57`	`'PARTIAL_TOKEN': ChoiceCompleter(("tokenP-a", "tokenP-b"), prefix='@', suffix='')`
`58`	`58`	`}`
`59`	`59`
`60`		`- # pylint: disable=arguments-differ,too-many-arguments`
	`60`	`+ # pylint: disable=arguments-differ,too-many-arguments,too-many-positional-arguments`
`61`	`61`	`def _construct_completer(self, expected_token=None, expected_tokens=None,`
`62`	`62`	`replaced_token=None, replaced_token_pos=-1,`
`63`	`63`	`completers=None, prefix="<!--"):`