chore: suppress new pylint lint

roypat · ShadowCurse · commit de0d42e81923 · 2024-12-11T12:12:36.000Z
With the devctr's python dependencies update, we pulled in a new pylint
version, which has a new `too-many-positional-arguments` lint. Fixing
this would be a significant refactor, so just suppress it, as it seems
low-value.

Signed-off-by: Patrick Roy &lt;roypat@amazon.co.uk&gt;
diff --git a/tests/integration_tests/style/test_python.py b/tests/integration_tests/style/test_python.py
@@ -37,7 +37,7 @@ def test_python_pylint():
         "fixme,too-many-instance-attributes,import-error,"
         "too-many-locals,too-many-arguments,consider-using-f-string,"
         "consider-using-with,implicit-str-concat,line-too-long,redefined-outer-name,"
-        "broad-exception-raised,duplicate-code tests tools .buildkite/*.py"
+        "broad-exception-raised,duplicate-code,too-many-positional-arguments tests tools .buildkite/*.py"
     )
     run(
         linter_cmd,

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ def test_python_pylint():`
`37`	`37`	`"fixme,too-many-instance-attributes,import-error,"`
`38`	`38`	`"too-many-locals,too-many-arguments,consider-using-f-string,"`
`39`	`39`	`"consider-using-with,implicit-str-concat,line-too-long,redefined-outer-name,"`
`40`		`- "broad-exception-raised,duplicate-code tests tools .buildkite/*.py"`
	`40`	`+ "broad-exception-raised,duplicate-code,too-many-positional-arguments tests tools .buildkite/*.py"`
`41`	`41`	`)`
`42`	`42`	`run(`
`43`	`43`	`linter_cmd,`