Reserve 'too-many-positional' / 'R0917' for later use (#9278)

See astral-sh/ruff#8946 Refs #9099
pylint-dev · Dec 13, 2023 · 278de0f · 278de0f
1 parent b1d8b4c
commit 278de0f
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 0 deletions.
diff --git a/doc/data/messages/t/too-many-positional/details.rst b/doc/data/messages/t/too-many-positional/details.rst
@@ -0,0 +1 @@
+Reserved message name, not yet implemented.
diff --git a/doc/data/messages/t/too-many-positional/related.rst b/doc/data/messages/t/too-many-positional/related.rst
@@ -0,0 +1,2 @@
+- `Ruff discussion <https://github.com/astral-sh/ruff/issues/8946>`_
+- `Pylint issue <https://github.com/pylint-dev/pylint/issues/9099>`_
diff --git a/doc/user_guide/checkers/features.rst b/doc/user_guide/checkers/features.rst
@@ -431,6 +431,10 @@ Design checker Messages
   simpler (and so easier to use) class.
 :too-many-locals (R0914): *Too many local variables (%s/%s)*
   Used when a function or method has too many local variables.
+:too-many-positional (R0917): *Too many positional arguments in a function call.*
+  Will be implemented in https://github.com/pylint-
+  dev/pylint/issues/9099,msgid/symbol pair reserved for compatibility with
+  ruff, see https://github.com/astral-sh/ruff/issues/8946.
 :too-many-public-methods (R0904): *Too many public methods (%s/%s)*
   Used when class has too many public methods, try to reduce this to get a
   simpler (and so easier to use) class.

diff --git a/doc/user_guide/messages/messages_overview.rst b/doc/user_guide/messages/messages_overview.rst
@@ -532,6 +532,7 @@ All messages in the refactor category:
    refactor/too-many-instance-attributes
    refactor/too-many-locals
    refactor/too-many-nested-blocks
+   refactor/too-many-positional
    refactor/too-many-public-methods
    refactor/too-many-return-statements
    refactor/too-many-statements

diff --git a/pylint/checkers/design_analysis.py b/pylint/checkers/design_analysis.py
@@ -81,6 +81,13 @@
         "too-many-boolean-expressions",
         "Used when an if statement contains too many boolean expressions.",
     ),
+    "R0917": (
+        "Too many positional arguments in a function call.",
+        "too-many-positional",
+        "Will be implemented in https://github.com/pylint-dev/pylint/issues/9099,"
+        "msgid/symbol pair reserved for compatibility with ruff, "
+        "see https://github.com/astral-sh/ruff/issues/8946.",
+    ),
 }
 SPECIAL_OBJ = re.compile("^_{2}[a-z]+_{2}$")
 DATACLASSES_DECORATORS = frozenset({"dataclass", "attrs"})
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		- `Ruff discussion <https://github.com/astral-sh/ruff/issues/8946>`_
		- `Pylint issue <https://github.com/pylint-dev/pylint/issues/9099>`_