From 01c559ba630618e1adbaa551c134419137df9b7b Mon Sep 17 00:00:00 2001 From: Pierre Sassoulas Date: Tue, 24 Sep 2024 12:53:28 +0200 Subject: [PATCH] [too-many-positional-arguments doc] Add the main option to use, real example --- .../messages/t/too-many-positional-arguments/bad.py | 9 +++++---- .../messages/t/too-many-positional-arguments/good.py | 10 ++++++---- .../messages/t/too-many-positional-arguments/pylintrc | 4 ++-- 3 files changed, 13 insertions(+), 10 deletions(-) diff --git a/doc/data/messages/t/too-many-positional-arguments/bad.py b/doc/data/messages/t/too-many-positional-arguments/bad.py index 2a2c5b4cac..d029bb4739 100644 --- a/doc/data/messages/t/too-many-positional-arguments/bad.py +++ b/doc/data/messages/t/too-many-positional-arguments/bad.py @@ -1,5 +1,6 @@ -class FiveArgumentMethods: - """The max positional arguments default is 5.""" +# +1: [too-many-positional-arguments] +def calculate_drag_force(velocity, area, density, drag_coefficient): + return 0.5 * drag_coefficient * density * area * velocity**2 - def take_five_args(self, a, b, c, d, e): # [too-many-positional-arguments] - pass + +drag_force = calculate_drag_force(30, 2.5, 1.225, 0.47) diff --git a/doc/data/messages/t/too-many-positional-arguments/good.py b/doc/data/messages/t/too-many-positional-arguments/good.py index ab4a9f2735..0308509acb 100644 --- a/doc/data/messages/t/too-many-positional-arguments/good.py +++ b/doc/data/messages/t/too-many-positional-arguments/good.py @@ -1,5 +1,7 @@ -class FiveArgumentMethods: - """The max positional arguments default is 5.""" +def calculate_drag_force(*, velocity, area, density, drag_coefficient): + return 0.5 * drag_coefficient * density * area * velocity**2 - def take_five_args(self, a, b, c, d, *, e=False): - pass + +drag_force = calculate_drag_force( + velocity=30, area=2.5, density=1.225, drag_coefficient=0.47 +) diff --git a/doc/data/messages/t/too-many-positional-arguments/pylintrc b/doc/data/messages/t/too-many-positional-arguments/pylintrc index 271f58965b..dceb2066b7 100644 --- a/doc/data/messages/t/too-many-positional-arguments/pylintrc +++ b/doc/data/messages/t/too-many-positional-arguments/pylintrc @@ -1,2 +1,2 @@ -[MESSAGES CONTROL] -disable=too-many-arguments +[DESIGN] +max-positional-arguments=3