Skip to content

Commit

Permalink
pylint: ignore too-many-positional-arguments
Browse files Browse the repository at this point in the history
Pylint 3.3.0 includes a new warning: too-many-positional-arguments [1].
We already ignore a similar too-many-arguments warning in several places
since "fixing" it is backward-incompatible.

1. pylint-dev/pylint#9099
  • Loading branch information
DifferentialOrange committed Sep 23, 2024
1 parent 61e96fc commit 938db75
Show file tree
Hide file tree
Showing 7 changed files with 10 additions and 10 deletions.
2 changes: 1 addition & 1 deletion tarantool/__init__.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
"""
This package provides API for interaction with a Tarantool server.
"""
# pylint: disable=too-many-arguments
# pylint: disable=too-many-arguments,too-many-positional-arguments

from tarantool.connection import Connection
from tarantool.mesh_connection import MeshConnection
Expand Down
2 changes: 1 addition & 1 deletion tarantool/connection.py
Original file line number Diff line number Diff line change
Expand Up @@ -812,7 +812,7 @@ def __init__(self,
.. _mp_bin: https://github.com/msgpack/msgpack/blob/master/spec.md#bin-format-family
.. _mp_array: https://github.com/msgpack/msgpack/blob/master/spec.md#array-format-family
"""
# pylint: disable=too-many-arguments,too-many-locals,too-many-statements
# pylint: disable=too-many-arguments,too-many-locals,too-many-statements,too-many-positional-arguments

if host is None and port is None and socket_fd is None:
raise ConfigurationError("need to specify host/port, "
Expand Down
2 changes: 1 addition & 1 deletion tarantool/connection_pool.py
Original file line number Diff line number Diff line change
Expand Up @@ -482,7 +482,7 @@ def __init__(self,
.. _box.info.status:
.. _box.info: https://www.tarantool.io/en/doc/latest/reference/reference_lua/box_info/
"""
# pylint: disable=too-many-arguments,too-many-locals
# pylint: disable=too-many-arguments,too-many-locals,too-many-positional-arguments

if not isinstance(addrs, list) or len(addrs) == 0:
raise ConfigurationError("addrs must be non-empty list")
Expand Down
2 changes: 1 addition & 1 deletion tarantool/mesh_connection.py
Original file line number Diff line number Diff line change
Expand Up @@ -463,7 +463,7 @@ def __init__(self,
:class:`~tarantool.Connection` exceptions,
:class:`~tarantool.MeshConnection.connect` exceptions
"""
# pylint: disable=too-many-arguments,too-many-locals
# pylint: disable=too-many-arguments,too-many-locals,too-many-positional-arguments

if addrs is None:
addrs = []
Expand Down
8 changes: 4 additions & 4 deletions tarantool/request.py
Original file line number Diff line number Diff line change
Expand Up @@ -269,7 +269,7 @@ def __init__(self, conn, salt, user, password, auth_type=AUTH_TYPE_CHAP_SHA1):
:param auth_type: Refer to :paramref:`~tarantool.Connection.auth_type`.
:type auth_type: :obj:`str`, optional
"""
# pylint: disable=too-many-arguments
# pylint: disable=too-many-arguments,too-many-positional-arguments

super().__init__(conn)

Expand Down Expand Up @@ -405,7 +405,7 @@ def __init__(self, conn, space_no, index_no, key, offset, limit, iterator):
:raise: :exc:`~AssertionError`
"""
# pylint: disable=too-many-arguments
# pylint: disable=too-many-arguments,too-many-positional-arguments

super().__init__(conn)
request_body = self._dumps({IPROTO_SPACE_ID: space_no,
Expand Down Expand Up @@ -446,7 +446,7 @@ def __init__(self, conn, space_no, index_no, key, op_list):
:raise: :exc:`~AssertionError`
"""
# pylint: disable=too-many-arguments
# pylint: disable=too-many-arguments,too-many-positional-arguments

super().__init__(conn)

Expand Down Expand Up @@ -569,7 +569,7 @@ def __init__(self, conn, space_no, index_no, tuple_value, op_list):
:raise: :exc:`~AssertionError`
"""
# pylint: disable=too-many-arguments
# pylint: disable=too-many-arguments,too-many-positional-arguments

super().__init__(conn)

Expand Down
2 changes: 1 addition & 1 deletion test/suites/lib/tarantool_server.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ class TarantoolServer():
"""
Class to start up a new Tarantool server.
"""
# pylint: disable=too-many-instance-attributes,too-many-arguments,duplicate-code
# pylint: disable=too-many-instance-attributes,too-many-arguments,duplicate-code,too-many-positional-arguments

default_tarantool = {
"bin": "tarantool",
Expand Down
2 changes: 1 addition & 1 deletion test/suites/test_ssl.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ def is_test_ssl():


class SslTestCase:
# pylint: disable=too-few-public-methods,too-many-instance-attributes,too-many-arguments
# pylint: disable=too-few-public-methods,too-many-instance-attributes,too-many-arguments,too-many-positional-arguments

def __init__(self,
name="",
Expand Down

0 comments on commit 938db75

Please sign in to comment.